-
-
Notifications
You must be signed in to change notification settings - Fork 556
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
docs: Document how to use GH workflows in forks #821
Merged
Merged
Changes from 31 commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
5a056a4
docs: Document how to use GH workflows in forks.
MaxymVlasov f257d06
f
MaxymVlasov ed67310
t
MaxymVlasov 56198ec
f
MaxymVlasov c16ad6d
f
MaxymVlasov f384e30
test != push only
MaxymVlasov bc92cb2
t
MaxymVlasov 6aaa5e9
f
MaxymVlasov 5d39fe1
f
MaxymVlasov 5e22eb0
f
MaxymVlasov 1d308c5
f
MaxymVlasov 582deea
Apply suggestions from code review
MaxymVlasov 41d3261
Replace to allowed-skips
MaxymVlasov 7417f7e
f
MaxymVlasov 4e34188
Try to fix "Unrecognized named-value: 'env'"
MaxymVlasov b55059d
fix issue with env visability in if's
MaxymVlasov b604547
Probably needs expansion
MaxymVlasov 01951e2
test
MaxymVlasov dc743f2
f
MaxymVlasov a09b8ad
empty commit to check how requirement checks actually work
MaxymVlasov 2c8e954
Clarify why skips allowed
MaxymVlasov e93151d
Merge branch 'master' into docs/gha_workflows_self_tests
MaxymVlasov 772cf9a
Rollback "skip" check
MaxymVlasov 927aa49
Merge branch 'master' into docs/gha_workflows_self_tests
MaxymVlasov 4ce1d19
Add check type in step name
MaxymVlasov e88989c
Simplify name and allow skips to not see so many skiped jobs
MaxymVlasov 60eee56
Make name difference invisible for end users
MaxymVlasov c681a6b
Document why they skipped
MaxymVlasov 4581cfa
Merge branch 'master' into docs/gha_workflows_self_tests
MaxymVlasov 8369cd1
Update .github/workflows/ci-cd.yml
MaxymVlasov 71e6a5a
Apply suggestions from code review
MaxymVlasov 9f125f3
Merge branch 'master' into docs/gha_workflows_self_tests
MaxymVlasov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like this, tbh
Does it make sense to move that
if
from here (build
) totests
andlint
just to see 1 less skipped job with very odd name?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's nothing you can do about it. Tell GH to fix it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
https://github.com/orgs/community/discussions/152293