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

Workflow optimisations for PR Checks action #497

Closed
deyanzz opened this issue Sep 1, 2023 · 0 comments · Fixed by #502
Closed

Workflow optimisations for PR Checks action #497

deyanzz opened this issue Sep 1, 2023 · 0 comments · Fixed by #502
Assignees
@deyanzz
Labels
Improvement Code changes driven by non business requirements
Milestone
0.16.0

Comments

@deyanzz
Copy link
Contributor

deyanzz commented Sep 1, 2023 •
edited

Problem

PR Checks action needs to be optimised, because currently they take about 1 hour to complete and pass successfully.
Sample PR Check.

Solution

To reduce the YAML script and remove some of the unnecessary steps of the pipeline actions during the PR process:

  • CMake Build (Release)
  • Start CTest suite (Release)
  • Attach Artifact

Alternatives

No response
@deyanzz deyanzz self-assigned this Sep 1, 2023
@deyanzz deyanzz added the Improvement Code changes driven by non business requirements label Sep 1, 2023
@deyanzz deyanzz linked a pull request Sep 8, 2023 that will close this issue
Workflow optimisations for PR Checks action #502
Merged
2 tasks
@deyanzz deyanzz mentioned this issue Sep 8, 2023
Merged
2 tasks
@deyanzz deyanzz added this to the 0.16.0 milestone Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@deyanzz deyanzz

Labels
Improvement Code changes driven by non business requirements
Projects
None yet
Milestone
0.16.0
Development

Successfully merging a pull request may close this issue.

Workflow optimisations for PR Checks action hashgraph/hedera-sdk-cpp
1 participant
@deyanzz