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

Enable URL validation #885

Merged
merged 3 commits into from
Feb 17, 2025
Merged

Enable URL validation #885

merged 3 commits into from
Feb 17, 2025

Conversation

fincd-aws
Copy link
Contributor

Turn on ExternalDomain check during PR workflow.

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Turn on ExternalDomain check during PR workflow.
@fincd-aws fincd-aws requested a review from a team as a code owner February 17, 2025 23:38
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-885.d3rijirjvbh87e.amplifyapp.com

set continue-on-error, now that Vale completes on 2 rules.
@fincd-aws fincd-aws assigned fincd-aws and geoffcline and unassigned fincd-aws Feb 17, 2025
@geoffcline
Copy link
Contributor

love it 10/10

@geoffcline geoffcline merged commit 2b91a2d into mainline Feb 17, 2025
3 checks passed
@fincd-aws fincd-aws deleted the fincd-vale-urls branch February 17, 2025 23:58
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