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

Add missing definitions to Dagger CUE API docs page #1800

Merged
merged 1 commit into from
Mar 16, 2022
Merged

Add missing definitions to Dagger CUE API docs page #1800

merged 1 commit into from
Mar 16, 2022

Conversation

gerhard
Copy link
Member

@gerhard gerhard commented Mar 14, 2022

Also links to the source of truth cue files, in the dagger/dagger repo.

While at it, move this page from Knowledge Base, which is a generic category, to Core Concepts. The URL didn't change so that we don't break existing links. FWIW /1213/dagger-cue would have been my choice.

Initially contributed as a change part of #1758

Related to #1768

@netlify
Copy link

netlify bot commented Mar 14, 2022

✔️ Deploy Preview for devel-docs-dagger-io ready!

🔨 Explore the source changes: dcf3b7e

🔍 Inspect the deploy log: https://app.netlify.com/sites/devel-docs-dagger-io/deploys/623108cda6dcc10009cf41a6

😎 Browse the preview: https://deploy-preview-1800--devel-docs-dagger-io.netlify.app

@gerhard gerhard mentioned this pull request Mar 15, 2022
3 tasks
@gerhard
Copy link
Member Author

gerhard commented Mar 15, 2022

@shykes another nudge for your 🏔

I have a few more related PRs but hate building inventories.

This made me want auto-merging PRs. Imagine setting this to auto-merge in e.g. 48 hours. Either someone reviews and merges first, or it gets auto-merged when the timer expires. WDYT? 🤓

Link to the source of truth cue files.

While at it, move this page from Knowledge Base, which is a generic
category, to Core Concepts. The URL didn't change so that we don't break
existing links. FWIW /1213/dagger-cue would have been my choice.

Initially contributed as a change part of
#1758

Should be merged after #1799

Related to #1768

Signed-off-by: Gerhard Lazu <gerhard@lazu.co.uk>
@gerhard
Copy link
Member Author

gerhard commented Mar 15, 2022

@shykes I rebased this on top of main, the comparison is:

BEFORE https://docs.dagger.io/1213/api/

AFTER https://deploy-preview-1800--devel-docs-dagger-io.netlify.app/1213/api

@shykes shykes merged commit 4e6d5d4 into dagger:main Mar 16, 2022
@gerhard gerhard deleted the add-missing-entries-to-dagger-cue-api branch March 23, 2022 07:51
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.

None yet

2 participants