flagsmith
flagsmith copied to clipboard
docs: Update doc to add commonLabels and commonAnnotations
Thanks for submitting a PR! Please check the boxes below:
- [x] I have run
pre-commit
to check linting - [x] I have added information to
docs/
if required so people know about the feature! - [x] I have filled in the "Changes" section below?
- [x] I have filled in the "How did you test this code" section below?
- [x] I have used a Conventional Commit title for this Pull Request
Changes
Updating docs according to the changes made in this PR
How did you test this code?
No tests, as it's just a docs update. The change itself was tested, see its PR.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
docs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Feb 21, 2024 1:35pm |
Someone is attempting to deploy a commit to the Flagsmith Team on Vercel.
A member of the Team first needs to authorize it.
Can you check the table formatting with the precommit linter?
Can you check the table formatting with the precommit linter?
Sure, the pre-commit failed, but I think, it's unrelated.
This looks good to me, I've added a comment to the PR on the charts repo with a question but this can be merged once we're happy with the PR on the charts repo. Thanks @Viktorsubota !
Hey @matthewelwell, Sorry, I don't see any comments there.
Could you point me to the one you left? Thanks!
Hi @Viktorsubota, I think at the time I had forgotten to actually submit the review but I have done that now. Could you check the comment I left there?
@dabeeeenster, @matthewelwell, Hey guys, the charts PR got merged, can we merge this PR as well?