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

Fix broken links in Design Review Guide #5948

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

donnie-msft
Copy link
Contributor

Update links and remove the Design review workflow image which I couldn't find.

@donnie-msft donnie-msft requested a review from a team as a code owner August 2, 2024 22:33
@donnie-msft donnie-msft enabled auto-merge (squash) August 5, 2024 18:30
@donnie-msft
Copy link
Contributor Author

@nkolev92 it looks like this path doesn't trigger CI. Maybe you can merge it without that since it's docs.

@nkolev92
Copy link
Member

nkolev92 commented Aug 5, 2024

I think we might need to revert the change for md, or tweak it.

While I can merge this, I don't this is a good long term solution. Force merging is disabled by default.

cc @zivkan in case he has an idea:

paths:
exclude:
- '*.md'
.

@jeffkl
Copy link
Contributor

jeffkl commented Aug 6, 2024

PrivateDev was triggered either since the branch name doesn't start with dev-. Feel free to get rid of the path exclusion in pr.yml since we can't disable the required checks by path in GitHub.

@donnie-msft donnie-msft changed the base branch from dev to dev-donnie-msft-designReviewGuide August 6, 2024 04:23
@donnie-msft donnie-msft merged commit 9e15727 into dev-donnie-msft-designReviewGuide Aug 6, 2024
2 checks passed
@donnie-msft donnie-msft deleted the donnie-msft-patch-1 branch August 6, 2024 04:23
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.

3 participants