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

Add trimja-action #107

Merged
merged 1 commit into from
Oct 8, 2024
Merged

Add trimja-action #107

merged 1 commit into from
Oct 8, 2024

Conversation

elliotgoodrich
Copy link
Owner

Add trimja-action to install, setup, and use trimja to reduce the amount of work needed to do on CI.

We have to remove the npm run docs part as this depends on building part of the project, which may have been removed with trimja. We should add an option to trimja-action to be able to include additional affected entries and then we can pass prep-for-docs too.

@elliotgoodrich elliotgoodrich force-pushed the add-trimja-action branch 5 times, most recently from fd787ec to 3eae803 Compare October 8, 2024 19:55
Add trimja-action to install, setup, and use trimja to reduce the
amount of work needed to do on CI.

We have to remove the `npm run docs` part as this depends on building
part of the project, which may have been removed with trimja.  We should
add an option to trimja-action to be able to include additional affected
entries and then we can pass `prep-for-docs` too.
@elliotgoodrich elliotgoodrich merged commit 2328cb5 into main Oct 8, 2024
13 checks passed
@elliotgoodrich elliotgoodrich deleted the add-trimja-action branch October 8, 2024 20:04
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.

1 participant