Skip to content

added step to check make has been run or not #399

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

Merged
merged 2 commits into from
Jun 26, 2025

Conversation

swastik959
Copy link
Contributor

added step to check make has been run or not in the build workflow
solves #342

@swastik959 swastik959 requested a review from calebschoepp May 30, 2025 16:29
@bacongobbler
Copy link
Contributor

Wonderful! Looks like the test failed because some files are out of date. @swastik959 would you mind regenerating those files as well and adding them to this PR? That would be really appreciated. Thank you!

@swastik959
Copy link
Contributor Author

swastik959 commented May 30, 2025

Wonderful! Looks like the test failed because some files are out of date. @swastik959 would you mind regenerating those files as well and adding them to this PR? That would be really appreciated. Thank you!

@bacongobbler done

@swastik959 swastik959 requested a review from bacongobbler June 3, 2025 05:20
@swastik959
Copy link
Contributor Author

@bacongobbler please have a look

@swastik959 swastik959 force-pushed the make-check branch 2 times, most recently from d170a9e to 216f3af Compare June 21, 2025 06:44
@swastik959
Copy link
Contributor Author

@bacongobbler can you please trigger the CI

@bacongobbler
Copy link
Contributor

Rebased against main.

@swastik959
Copy link
Contributor Author

@bacongobbler the CI has passed please review it now and apologies for delay was caught up in a release 🙏

@bacongobbler bacongobbler merged commit b5817df into spinframework:main Jun 26, 2025
12 checks passed
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.

4 participants