Skip to content

chore: bump internal tooling to v0.14.0#8905

Merged
gerhard merged 4 commits into
dagger:mainfrom
sipsma:improve-releasing-during-v0.14.0
Nov 12, 2024
Merged

chore: bump internal tooling to v0.14.0#8905
gerhard merged 4 commits into
dagger:mainfrom
sipsma:improve-releasing-during-v0.14.0

Conversation

@sipsma
Copy link
Copy Markdown
Contributor

@sipsma sipsma commented Nov 8, 2024

Just autogenerated stuff here.

FTR, hit lots of things big and small but will split them out to follow ups so this isn't blocked too long:

Signed-off-by: Erik Sipsma <erik@sipsma.dev>
@sipsma sipsma requested review from gerhard and jedevc November 8, 2024 23:18
marcosnils and others added 2 commits November 8, 2024 15:33
- Rename `DEngine` to `DaggerEngine`
- Fix `module_test` to use the correct core function name

Signed-off-by: Marcos Lilljedahl <marcosnils@gmail.com>
Signed-off-by: Erik Sipsma <erik@sipsma.dev>
@helderco
Copy link
Copy Markdown
Contributor

helderco commented Nov 9, 2024

  • Fix python publish error (being handled by Helder)

Copy link
Copy Markdown
Contributor

@jedevc jedevc left a comment

Choose a reason for hiding this comment

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

LGTM as-is - can merge now, and we can do follow-ups later 🎉

@jedevc
Copy link
Copy Markdown
Contributor

jedevc commented Nov 12, 2024

  • Fix dynamic optional error hit during SDK publishes

Pain 😢

TIL: `/` only applies to git (not GitHub)

The semantics are: if a new PR gets created which changes the
`.changes/.next` file, deploy a preview environment.

These PRs **must** be opened from branches that belog to the upstream
repository - `dagger/dagger` - otherwise the secret will not be
available and therefore the `dagger/dagger.io` PR cannot be created due
to insufficient permissions. I have updated `RELEASING.md` to reflect
this.

Signed-off-by: Gerhard Lazu <gerhard@dagger.io>
Copy link
Copy Markdown
Contributor

@gerhard gerhard left a comment

Choose a reason for hiding this comment

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

I pushed the fix for daggerverse-preview & added extra details to RELEASING.md.

CI runners are ready for this, all looking good, merging 🧑‍🚒

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.

5 participants