Skip to content

[SYNCOPE-1738] Refactoring Reports#421

Merged
ilgrosso merged 8 commits intoapache:3_0_Xfrom
ilgrosso:SYNCOPE-1738
Mar 9, 2023
Merged

[SYNCOPE-1738] Refactoring Reports#421
ilgrosso merged 8 commits intoapache:3_0_Xfrom
ilgrosso:SYNCOPE-1738

Conversation

@ilgrosso
Copy link
Copy Markdown
Member

@ilgrosso ilgrosso commented Mar 7, 2023

  • Finalize Console
  • Update Docs

* @throws JobExecutionException if anything goes wrong
*/
protected abstract String doExecute(
boolean dryRun, OutputStream os, String executor, JobExecutionContext context)

Check notice

Code scanning / CodeQL

Useless parameter

The parameter 'executor' is never used.
* @throws JobExecutionException if anything goes wrong
*/
protected abstract String doExecute(
boolean dryRun, OutputStream os, String executor, JobExecutionContext context)

Check notice

Code scanning / CodeQL

Useless parameter

The parameter 'context' is never used.
@ilgrosso ilgrosso marked this pull request as ready for review March 8, 2023 16:15
@ilgrosso ilgrosso merged commit 4d92b4c into apache:3_0_X Mar 9, 2023
@ilgrosso ilgrosso deleted the SYNCOPE-1738 branch March 9, 2023 07:34
ilgrosso added a commit that referenced this pull request Mar 9, 2023
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.

2 participants