-
Notifications
You must be signed in to change notification settings - Fork 4
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
Writing custom check functions vignette #127
base: main
Are you sure you want to change the base?
Conversation
Merge branch 'main' into ak/custom-fn-dev-article/121 # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
echo 'dir=./docs/dev' >> $GITHUB_OUTPUT | ||
else | ||
echo 'dir=./docs' >> $GITHUB_OUTPUT | ||
fi | ||
- name: Deploy PR preview to Netlify |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These changes fix broken dev previews URLs resulting from the fact we've got both release and dev versions of docs now
check_for_errors(validate_submission(hub_path, file_path)), | ||
check_for_errors(validate_submission( | ||
hub_path, file_path, | ||
skip_submit_window_check = TRUE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just fixing an oversight in a previously added test that was now returning a submission window check error
This PR adds:
I've also pulled out information that is useful to both the writing and deploying custom functions vignette into children documents which are now deployed in both. Because of this it might be more useful to have the rendered vignette open to see the content in place.