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

Build the project schema model before the configuration phase #29153

Merged
merged 1 commit into from
May 15, 2024

Conversation

jbartok
Copy link
Member

@jbartok jbartok commented May 15, 2024

Context

Contributor Checklist

  • Review Contribution Guidelines.
  • Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
  • Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
  • Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team.
  • Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective.
  • Provide unit tests (under <subproject>/src/test) to verify logic.
  • Update User Guide, DSL Reference, and Javadoc for public-facing changes.
  • Ensure that tests pass sanity check: ./gradlew sanityCheck.
  • Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest.

Reviewing cheatsheet

Before merging the PR, comments starting with

  • ❌ ❓must be fixed
  • 🤔 💅 should be fixed
  • 💭 may be fixed
  • 🎉 celebrate happy things

@jbartok jbartok added this to the 8.9 RC1 milestone May 15, 2024
@jbartok jbartok self-assigned this May 15, 2024
@jbartok jbartok requested review from a team as code owners May 15, 2024 08:52
@jbartok jbartok requested review from bamboo and mlopatkin and removed request for a team May 15, 2024 08:52
@jbartok jbartok force-pushed the jb/declarative-dsl/unique-project-schema branch from 7f9f65c to 682abd4 Compare May 15, 2024 08:57
Copy link
Member

@eskatos eskatos left a comment

Choose a reason for hiding this comment

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

Approved with one minor request.

@jbartok jbartok force-pushed the jb/declarative-dsl/unique-project-schema branch from 682abd4 to 4e5476d Compare May 15, 2024 09:06
@jbartok jbartok added this pull request to the merge queue May 15, 2024
Merged via the queue into master with commit 86a0620 May 15, 2024
22 checks passed
@jbartok jbartok deleted the jb/declarative-dsl/unique-project-schema branch May 15, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants