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

[docs-beta] Fix preview links in docs-revamp GH action #24046

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

hellendag
Copy link
Member

@hellendag hellendag commented Aug 29, 2024

Summary & Motivation

The glob isn't finding md/mdx files properly. Not sure why, but if I just cd into the docs directory and use syntax like in the old docs action, it works.

How I Tested These Changes

Modify a md file in docs-beta/docs, verify that the GH action posts a comment with the preview URL.

Changelog [New | Bug | Docs]

NOCHANGELOG

@hellendag hellendag force-pushed the dish/test-gh-action branch 5 times, most recently from 98ab33b to c0da124 Compare August 29, 2024 13:44
@dagster-io dagster-io deleted a comment from github-actions bot Aug 29, 2024
@hellendag hellendag force-pushed the dish/test-gh-action branch 2 times, most recently from 8f85dd5 to 51e3188 Compare August 29, 2024 13:51
@dagster-io dagster-io deleted a comment from github-actions bot Aug 29, 2024
@hellendag hellendag force-pushed the dish/test-gh-action branch from 51e3188 to 4adaf7e Compare August 29, 2024 13:56
@dagster-io dagster-io deleted a comment from github-actions bot Aug 29, 2024
@hellendag hellendag changed the title Test GH action [docs-beta] Fix preview links in docs-revamp GH action Aug 29, 2024
@hellendag hellendag marked this pull request as ready for review August 29, 2024 13:57
Copy link

Deploy preview for dagster-docs-beta ready!

Preview available at https://dagster-docs-beta-a1jyr0ti9-elementl.vercel.app

Direct link to changed pages:

@hellendag hellendag merged commit 7fb590e into master Aug 29, 2024
2 checks passed
@hellendag hellendag deleted the dish/test-gh-action branch August 29, 2024 14:39
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.

2 participants