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.

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

Skip spotless tasks in Github actions to accelerate the CI #1021

Merged

Conversation

lijamie98
Copy link
Collaborator

@lijamie98 lijamie98 commented Aug 1, 2023

PR Checklist

PR Structure

  • This PR has reasonably narrow scope (if not, break it down into smaller PRs).
  • This PR avoids mixing refactoring changes with feature changes (split into two PRs
    otherwise).
  • This PR's title starts with name of package that is most changed in the PR, ex.
    paymentservice.stellar, or all or doc if the changes are broad or impact many
    packages.

Thoroughness

  • This PR adds tests for the most critical parts of the new functionality or fixes.

What

Skip spotless tasks in Github actions to accelerate the CI

Why

Optimization

@stellar-jenkins
Copy link

Reference Server Preview is available here:
https://anchor-ref-pr1021.previews.kube001.services.stellar-ops.com/
SEP Server Preview is available here:
https://anchor-sep-pr1021.previews.kube001.services.stellar-ops.com/

@lijamie98 lijamie98 force-pushed the feature/skip-spotless-in-GH-actions branch from ab434bc to 6d592e1 Compare August 1, 2023 07:22
@stellar-jenkins
Copy link

Reference Server Preview is available here:
https://anchor-ref-pr1021.previews.kube001.services.stellar-ops.com/
SEP Server Preview is available here:
https://anchor-sep-pr1021.previews.kube001.services.stellar-ops.com/

@lijamie98 lijamie98 force-pushed the feature/skip-spotless-in-GH-actions branch from 6d592e1 to ac94812 Compare August 1, 2023 07:29
@stellar-jenkins
Copy link

Reference Server Preview is available here:
https://anchor-ref-pr1021.previews.kube001.services.stellar-ops.com/
SEP Server Preview is available here:
https://anchor-sep-pr1021.previews.kube001.services.stellar-ops.com/

@lijamie98 lijamie98 force-pushed the feature/skip-spotless-in-GH-actions branch from ac94812 to 01f3716 Compare August 1, 2023 07:39
@stellar-jenkins
Copy link

Reference Server Preview is available here:
https://anchor-ref-pr1021.previews.kube001.services.stellar-ops.com/
SEP Server Preview is available here:
https://anchor-sep-pr1021.previews.kube001.services.stellar-ops.com/

@@ -47,7 +51,7 @@ jobs:
- name: Gradle test and build. (unit tests, integration tests, end-2-end tests and build)
env:
run_docker: false
run: ./gradlew clean build --no-daemon --stacktrace
run: ./gradlew clean build --no-daemon --stacktrace -x spotlessApply -x spotlessCheck -x spotlessKotlin -x spotlessKotlinApply -x javadoc -x javadocJar -x sourcesJar
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can skip spotlessApply, but not check. If a developer tries to submit unformatted code, we should be catching it in CI.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@philipliu If spotlessCheck fails, does it fail the CI?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed -x spotlessCheck

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I missed this earlier, but we should keep spotlessKotlin too right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem at all. Thanks for catching these gotchas. I had removed -x spotlessKotlin.

@lijamie98 lijamie98 force-pushed the feature/skip-spotless-in-GH-actions branch from 01f3716 to 7a9de59 Compare August 2, 2023 12:43
@stellar-jenkins
Copy link

Reference Server Preview is available here:
https://anchor-ref-pr1021.previews.kube001.services.stellar-ops.com/
SEP Server Preview is available here:
https://anchor-sep-pr1021.previews.kube001.services.stellar-ops.com/

Copy link
Contributor

@philipliu philipliu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a minor comment about spotlessKotlin.

@lijamie98 lijamie98 self-assigned this Aug 7, 2023
@lijamie98 lijamie98 force-pushed the feature/skip-spotless-in-GH-actions branch from 799d37f to a540f89 Compare August 7, 2023 02:41
@stellar-jenkins
Copy link

Reference Server Preview is available here:
https://anchor-ref-pr1021.previews.kube001.services.stellar-ops.com/
SEP Server Preview is available here:
https://anchor-sep-pr1021.previews.kube001.services.stellar-ops.com/

@stellar-jenkins
Copy link

Something went wrong with PR preview build please check

@stellar-jenkins
Copy link

Reference Server Preview is available here:
https://anchor-ref-pr1021.previews.kube001.services.stellar-ops.com/
SEP Server Preview is available here:
https://anchor-sep-pr1021.previews.kube001.services.stellar-ops.com/

@lijamie98 lijamie98 merged commit 5e844a3 into stellar:develop Aug 8, 2023
5 checks passed
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