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

Set project name #365

Merged
merged 2 commits into from
May 21, 2024
Merged

Set project name #365

merged 2 commits into from
May 21, 2024

Conversation

twyatt
Copy link
Member

@twyatt twyatt commented May 17, 2024

Builds would always show the following warning:

Project accessors enabled, but root project name not explicitly set for 'krayon'. Checking out the project in different folders will impact the generated code and implicitly the buildscript classpath, breaking caching.

This resolves the issue by explicitly setting the project name.

@twyatt twyatt added the maintenance General maintenance that doesn't effect the public API. label May 17, 2024
@twyatt twyatt marked this pull request as ready for review May 17, 2024 08:46
@twyatt twyatt requested review from cedrickcooke and a team as code owners May 17, 2024 08:46
@twyatt twyatt requested a review from QuantumRand May 17, 2024 08:46
@github-actions GitHub Actions
Copy link

github-actions bot commented May 17, 2024 •
edited
Loading

Test Results

 62 files  ±0   62 suites  ±0   18s ⏱️ ±0s
321 tests ±0  321 ✅ ±0  0 💤 ±0  0 ❌ ±0 
366 runs  ±0  366 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit e68b1b6. ± Comparison against base commit e83b034.

♻️ This comment has been updated with latest results.

@twyatt twyatt enabled auto-merge (squash) May 19, 2024 03:08
@twyatt twyatt merged commit 697dcc3 into main May 21, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cedrickcooke cedrickcooke cedrickcooke approved these changes

@QuantumRand QuantumRand QuantumRand approved these changes

Assignees
No one assigned
Labels
maintenance General maintenance that doesn't effect the public API.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@twyatt @QuantumRand @cedrickcooke