Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix docs typo #1597

Merged
merged 2 commits into from
Nov 25, 2024
Merged

fix docs typo #1597

merged 2 commits into from
Nov 25, 2024

Conversation

CommanderStorm
Copy link
Collaborator

No description provided.

@CommanderStorm CommanderStorm enabled auto-merge (squash) November 25, 2024 11:48
Copy link
Member

@nyurik nyurik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx

docs/src/martin-cp.md Outdated Show resolved Hide resolved
@nyurik
Copy link
Member

nyurik commented Nov 25, 2024

CI currently excludes markdown-only tests to save on the runners. We probably should have some bypass in the CI so it thinks everything passed

@nyurik nyurik disabled auto-merge November 25, 2024 17:21
@nyurik nyurik merged commit 2cc5175 into main Nov 25, 2024
2 checks passed
@nyurik nyurik deleted the CommanderStorm-patch-1 branch November 25, 2024 17:21
@nyurik
Copy link
Member

nyurik commented Nov 25, 2024

P.S. Please submit PRs only from your own fork - this is a requirement for all contributors including the core ones. See https://gist.github.com/nyurik/4e299ad832fd2dd43d2b27191ed3ec30

@CommanderStorm
Copy link
Collaborator Author

Okay, will do for the future.

Btw: you/other maplibre Governing Board members can enforce this via a branch protection rule:

image

We should likely create a contributing guide which includes above, will steal the wording and come up with a first draft ^^

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants