Skip to content

Reuse generated code in E2E#237

Merged
janisz merged 5 commits intomasterfrom
reuse_jar
Mar 30, 2023
Merged

Reuse generated code in E2E#237
janisz merged 5 commits intomasterfrom
reuse_jar

Conversation

@janisz
Copy link
Copy Markdown
Contributor

@janisz janisz commented Mar 27, 2023

Since we generate API client in jenkins-plugin and upload jar to repository there is no need to regenerate client in e2e tests. We can reuse previously generated client. This will save build time, reduce number of dependencies and reuse plugin code giving us more coverage.

@janisz janisz force-pushed the reuse_jar branch 2 times, most recently from 0931489 to e2c58f2 Compare March 28, 2023 11:36
Comment thread functionaltest-jenkins-plugin/build.gradle
@janisz janisz requested a review from dhaus67 March 29, 2023 09:59
@github-advanced-security
Copy link
Copy Markdown

You have successfully added a new CodeQL configuration .github/workflows/tests.yaml:build. As part of the setup process, we have scanned this repository and found 1 existing alert. Please check the repository Security tab to see all alerts.

@janisz janisz changed the base branch from master to remove_custom_proxy March 29, 2023 10:40
This was referenced Mar 29, 2023
@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023

@janisz janisz force-pushed the remove_custom_proxy branch from efabb0c to 5b7336c Compare March 29, 2023 11:08
@janisz janisz changed the base branch from remove_custom_proxy to fix_codeql March 29, 2023 11:08
@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023

@janisz queued this pull request to merge with Graphite.

Base automatically changed from fix_codeql to master March 29, 2023 13:54
@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023

Graphite failed to merge this pull request for an unknown reason. Try your merge again, or report a bug if you see this consistently.

@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023

@janisz queued this pull request to merge with Graphite.

@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023

Graphite rebased this pull request as part of a merge.

@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023

Graphite failed to merge this pull request for an unknown reason. Try your merge again, or report a bug if you see this consistently.

@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 30, 2023

@janisz queued this pull request to merge with Graphite.

@janisz janisz merged commit 4696a28 into master Mar 30, 2023
@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 30, 2023

@janisz merged this pull request with Graphite.

@janisz janisz deleted the reuse_jar branch March 30, 2023 12:11
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.

3 participants