Skip to content
This repository has been archived by the owner on Jul 23, 2024. It is now read-only.

ci: Fail early if the git repo is dirty post build+test #260

Merged
merged 1 commit into from
Apr 19, 2023

Conversation

rgolangh
Copy link
Contributor

@rgolangh rgolangh commented Apr 19, 2023

Fail early if the project leaves the repo dirty post build and test.
This should be an indication of uncommited changes of the generated files

@rgolangh
Copy link
Contributor Author

rebasing

@gciavarrini
Copy link
Contributor

Can you update the PR description?

@eloycoto
Copy link
Collaborator

If you can add the gitignore the files on notification-service will be great.

Signed-off-by: Roy Golan <rgolan@redhat.com>
@rgolangh rgolangh changed the title update post action https://github.com/parodos-dev/parodos/pull/260 Apr 19, 2023
@rgolangh rgolangh changed the title https://github.com/parodos-dev/parodos/pull/260 ci: Fail early if the git repo is dirty post build+test Apr 19, 2023
@eloycoto
Copy link
Collaborator

/approve
/lgtm

@openshift-ci
Copy link

openshift-ci bot commented Apr 19, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: eloycoto

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pkliczewski
Copy link
Collaborator

/lgtm

@openshift-merge-robot openshift-merge-robot merged commit d3f945c into parodos-dev:main Apr 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants