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

ci: javadoc generation workflow logic can be improved #3525

Closed
ossdhaval opened this issue Jan 5, 2023 · 0 comments · Fixed by #3526
Closed

ci: javadoc generation workflow logic can be improved #3525

ossdhaval opened this issue Jan 5, 2023 · 0 comments · Fixed by #3526
Assignees
@ossdhaval
Labels
area-CI Issue or changes required in automatic builds or CI infrastructure effort-1 Relative effort required for completion of issue or PR priority-4 Minor issue or PR is not relevant to core functions, or relates to the usability of system triaged Issue or PR is fully triaged

Comments

@ossdhaval
Copy link
Contributor

Describe the bug

Multiple SED commands used at this line can be replaced by much simpler logic. Just use: sed 's/\/target\/site\/apidocs//'
@ossdhaval ossdhaval added area-CI Issue or changes required in automatic builds or CI infrastructure effort-1 Relative effort required for completion of issue or PR priority-4 Minor issue or PR is not relevant to core functions, or relates to the usability of system triaged Issue or PR is fully triaged labels Jan 5, 2023
@ossdhaval ossdhaval self-assigned this Jan 5, 2023
@ossdhaval ossdhaval mentioned this issue Jan 5, 2023
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ossdhaval ossdhaval

Labels
area-CI Issue or changes required in automatic builds or CI infrastructure effort-1 Relative effort required for completion of issue or PR priority-4 Minor issue or PR is not relevant to core functions, or relates to the usability of system triaged Issue or PR is fully triaged
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: remove multiple sed commands JanssenProject/jans
1 participant
@ossdhaval