Skip to content

Commit dist when changes occurred.#5

Merged
dhaus67 merged 6 commits intomainfrom
dh/commit-dist
Nov 7, 2023
Merged

Commit dist when changes occurred.#5
dhaus67 merged 6 commits intomainfrom
dh/commit-dist

Conversation

@dhaus67
Copy link
Copy Markdown

@dhaus67 dhaus67 commented Nov 3, 2023

Description

Enable committing changes to the dist directory on pushes to main.

The alternative is to have a check step which will detect if there's any changes to the dist file and require users to manually run npm run all locally + committing this before merging.

Successful action run: https://github.com/stackrox/central-login/actions/runs/6741999434/job/18327406312
Resulting commit: cad0503

@dhaus67 dhaus67 marked this pull request as ready for review November 3, 2023 06:29
@dhaus67 dhaus67 requested a review from janisz November 3, 2023 06:29
@dhaus67 dhaus67 mentioned this pull request Nov 3, 2023
Copy link
Copy Markdown
Contributor

@janisz janisz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need it?

@dhaus67
Copy link
Copy Markdown
Author

dhaus67 commented Nov 3, 2023

As said in the description of the PR, typescript girhub action expect this and I don't want to force people to manually do this and pollute their PRs with the code changes.

Base automatically changed from dh/init-commit to main November 7, 2023 16:06
@dhaus67 dhaus67 merged commit 9487c33 into main Nov 7, 2023
@dhaus67 dhaus67 deleted the dh/commit-dist branch November 7, 2023 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants