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.

Sign up for GitHub

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

Jump to bottom

Allow to specify report ids for new publish steps #186

Closed
8 tasks done
ErikRehmTT opened this issue Oct 8, 2024 · 0 comments · Fixed by #226
Closed
8 tasks done

Allow to specify report ids for new publish steps #186

ErikRehmTT opened this issue Oct 8, 2024 · 0 comments · Fixed by #226
Assignees
@MartinGroscheTT @MxEh-TT
Labels
feature New feature request

Comments

@ErikRehmTT
Copy link
Collaborator

ErikRehmTT commented Oct 8, 2024 •
edited
Loading

Description

As power user I want to execute the new publish steps only for certain reports. I can do so for ttGenerateReports and ttUploadReports already.

Acceptance Criteria

  • All new publish steps support the reportIds parameter as well
  • When reportIds is provided, the steps action is only applied to the given reports
  • In the default case (not provided) the steps action is applied to all reports
  • In case a given reportId does not exist, a warning is printed
  • All post action steps have a fail on error parameter
  • ttGenerateReports and ttUploadReports behave the same way as described above
  • snippet generator allows to set reportIds in the advanced section
  • additional docs contain the new parameters
@ErikRehmTT ErikRehmTT added the feature New feature request label Oct 8, 2024
@MartinGroscheTT MartinGroscheTT self-assigned this Feb 5, 2025
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Remove unnecessary imports (#186)
cb27f64
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Update reomove empty reportIds handling (#186)
5c8a7e9
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Add faileOnError property (#186)
96109ff
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Add reportId handling (#186)
3cc76c9
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Remove warning if no files found (#186)
dfca87e
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Refactor reports to logs (#186)
b73eb61
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Add tests for report ids and failOnError property (#186)
90de842
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Add tests for processReport (#186)
2d26979
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Add fail on Error property (#186)
93c1495
MartinGroscheTT added a commit that referenced this issue Feb 10, 2025
@MartinGroscheTT
Add fail on Error property (#186)
fc398e6
@MartinGroscheTT MartinGroscheTT mentioned this issue Feb 10, 2025
Merged
@MartinGroscheTT MartinGroscheTT linked a pull request Feb 10, 2025 that will close this issue
Allow to specify report ids for new publish steps #226
Merged
MartinGroscheTT added a commit that referenced this issue Feb 11, 2025
@MartinGroscheTT
Add failOnError false test (#186)
e018a91
MartinGroscheTT added a commit that referenced this issue Feb 11, 2025
@MartinGroscheTT
Update container tests for failOnError property (#186)
5706f60
MartinGroscheTT added a commit that referenced this issue Feb 11, 2025
@MartinGroscheTT
Update documentation for failOnError property (#186)
8492714 
- provideConfig
- uploadReport
- generateReport
MxEh-TT added a commit that referenced this issue Feb 25, 2025
@MxEh-TT
add review notes (#186)
c045ce1
MxEh-TT added a commit that referenced this issue Feb 25, 2025
@MxEh-TT
adjust advanced docs (#186)
8d0c721
MxEh-TT added a commit that referenced this issue Feb 25, 2025
@MxEh-TT
adjust tests to new logs (#186)
917f217
MxEh-TT added a commit that referenced this issue Mar 3, 2025
@MxEh-TT
add reportIds to jelly for snippet generation (#186)
664474c
MxEh-TT added a commit that referenced this issue Mar 3, 2025
@MxEh-TT
fix value in jelly (#186)
0924d9e
MxEh-TT added a commit that referenced this issue Mar 3, 2025
@MxEh-TT
adjust reportIds setter (#186)
b6db221
MxEh-TT added a commit that referenced this issue Mar 3, 2025
@MxEh-TT
update depCheckVersion (#186)
a3fd448
MxEh-TT added a commit that referenced this issue Mar 4, 2025
@MxEh-TT
add snippet test for strings (#186)
3553951
MxEh-TT added a commit that referenced this issue Mar 4, 2025
@MxEh-TT
add review notes (#186)
5a61cad 
- add missing code snippet generator to upload step
- fix triming reportIds
- adjust tests for trimming of strings
MxEh-TT added a commit that referenced this issue Mar 4, 2025
@MxEh-TT
adjust memory settings (#186)
519e7de
MxEh-TT pushed a commit that referenced this issue Mar 4, 2025
@MartinGroscheTT
Allow to specify report ids for new publish steps (#186)
09353cb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MartinGroscheTT MartinGroscheTT

@MxEh-TT MxEh-TT

Labels
feature New feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow to specify report ids for new publish steps jenkinsci/ecu-test-execution-plugin
3 participants
@MartinGroscheTT @ErikRehmTT @MxEh-TT