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

Issues #11

Merged
merged 9 commits into from
Jul 9, 2021
Merged

Issues #11

merged 9 commits into from
Jul 9, 2021

Conversation

nigelgbanks
Copy link
Collaborator

Resolves issues #9 and #10, and makes changes that will allow us to get the plugin hosted at https://plugins.gradle.org/. This will allow us to use build scans, and should make builds slightly faster.

Resolved some complications issues.

The kotlin plugin in intellij is not compatiable with gradle plugin
projects using the kotlin-dsl gradle plugin.
We can't reliably assume that the image/tag actually corresponds
to the same inputs when switch between changes we already built before.

As gradle happy places the generated digest.json file and ignores that
the tagged image hasn't actually been changed.

This is because gradle isn't actually tracking the generated image but
rather its approximate digest. So the tag can point to the previous
build when switching between two sets of changes for which gradle has
a cached output.
@nigelgbanks
Copy link
Collaborator Author

I've tested the changes against the main branch of buildkit all seems well.

@nigelgbanks nigelgbanks added bug Something isn't working enhancement New feature or request labels Jul 8, 2021
@nigelgbanks nigelgbanks marked this pull request as ready for review July 9, 2021 11:00
Copy link
Member

@dannylamb dannylamb left a comment

Choose a reason for hiding this comment

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

I'm no kotlin 🧙 , but it works 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants