-
Notifications
You must be signed in to change notification settings - Fork 747
Workflow output definition #4784
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
Changes from 20 commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
0de777a
Add initial prototype
bentsherman 14beedc
Fix race condition
bentsherman 062b421
Simplify output DSL
bentsherman 06a5872
Support default publish options in path method
bentsherman eecf619
Rename OutputDsl -> WorkflowPublishDsl
bentsherman b238a81
Replace process selector with channel/topic selectors, add e2e test
bentsherman d0fa980
cleanup PublishOp
bentsherman 8279743
Add topic operator (not working)
bentsherman 7681e2d
Merge branch 'master' into 4670-workflow-outputs
pditommaso 88d7ead
Add topic op test
pditommaso 1e3d539
clean up e2e test
bentsherman 925bc67
Fix issue with topic operator
bentsherman e4608b3
Apply suggestions from review
bentsherman 1e69f3e
Add output directory option to CLI, config, output DSL
bentsherman 01b0570
Validate publish options
bentsherman c43afad
Update docs
bentsherman 570da27
Add defaults to directory statement
bentsherman 0ad12eb
Update docs
bentsherman 3e0823b
Apply suggestions from review
bentsherman 5a1a7bb
Fix workflow binding
bentsherman 3bdd0fe
Fix dynamic path name
bentsherman 59d83c0
Apply suggestions from review
bentsherman 5beca46
Merge branch 'master' into 4670-workflow-outputs
pditommaso b67376c
Add `publish:` section to process
bentsherman 65d9111
Update docs
bentsherman 35df924
Add publish options to output DSL
bentsherman 509f271
Change publish op to handle multiple task dirs
bentsherman 6fcbf6f
Fix error when no output block is specified
bentsherman 550766a
Rename output -> publish, rule -> target
bentsherman 72d8a6f
Disallow absolute path in publish target
bentsherman 370aa0b
Add feature flag
bentsherman 76af5c9
Allow multi-channel output to be published
bentsherman 0833e11
Add overwrite modes for deep / lenient / standard hash comparison
bentsherman 4614f26
Merge branch 'master' into 4670-workflow-outputs
pditommaso 02745f7
Factor out HashBuilder from CacheHelper
bentsherman 0583b3d
Add index file definition
bentsherman e60403d
Fix failing tests
bentsherman 12c2720
Apply suggestions from review
bentsherman d998878
Don't write index file if no records were published
bentsherman 95a110a
Redirect to `null` to disable publishing
bentsherman 7953e79
Remove ternary hack, require parentheses instead
bentsherman 856209b
Replace publish path option with ability to reroute targets in publis…
bentsherman 50484d9
Apply suggestions from review
bentsherman ab9118e
Merge branch 'master' into 4670-workflow-outputs
pditommaso 84e6382
Merge branch 'master' into 4670-workflow-outputs
pditommaso 2b80f47
Merge branch 'master' into 4670-workflow-outputs
pditommaso d504e45
Minor change [ci skip]
pditommaso 6185d84
Apply suggestions from review
bentsherman 79cc68c
Fix typo [ci skip]
pditommaso a70af4e
Merge branch 'master' into 4670-workflow-outputs
pditommaso ef4305d
Add shorthand for publishing single file to index
bentsherman b8cf823
Fold PublishIndexOp into PublishOp, add test for OutputDsl,
bentsherman 02ba0c7
Update index file default for single file
bentsherman 0db73c8
Use file base name for default index
bentsherman 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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
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.
Uh oh!
There was an error while loading. Please reload this page.