Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Grunt action to verify if compiled files are up to date #261

Merged
merged 2 commits into from
Nov 23, 2023

Conversation

csidirop
Copy link
Contributor

This PR adds a GH workflow to check whether the compiled files are up to date with their sources.

It can easily happen that we commit code without grunt running, or merge a PR that is not up to date with master. This GH workflow causes the test to fail in such a case and you can do it afterwards.

Verifies compiled files are up to date

Signed-off-by: Christos Sidiropoulos <[email protected]>
@beatrycze-volk beatrycze-volk added the 🛠️ maintenance A task to keep the code up-to-date and manageable. label Nov 23, 2023
@beatrycze-volk beatrycze-volk added this to the DFG-Viewer 6.0.0 milestone Nov 23, 2023
@beatrycze-volk
Copy link
Contributor

Thanks for your work!

Could you updated this PR so the newly introduced check actually pass?

Signed-off-by: Christos Sidiropoulos <[email protected]>
@csidirop
Copy link
Contributor Author

I had actually left the files as they were and not updated them so that you can see the case directly when it fails :D

But updated it now!

@csidirop
Copy link
Contributor Author

csidirop commented Nov 23, 2023

One more thing: I have set node.js to version 12, but you can add more versions to test additionally. (eg. node-version: [12.x, 14.x, 16.x]) In my tests this did not lead to any different results.

@beatrycze-volk beatrycze-volk merged commit f498e8d into slub:master Nov 23, 2023
5 checks passed
@csidirop csidirop deleted the gruntaction branch November 23, 2023 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️ maintenance A task to keep the code up-to-date and manageable.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants