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.

Sign up for GitHub

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

Jump to bottom

Add new builder-tools tag chain-index #584

Merged
merged 4 commits into from
Apr 6, 2022
Merged

Add new builder-tools tag chain-index #584

merged 4 commits into from
Apr 6, 2022

Conversation

fill-the-fill
Copy link
Collaborator

  • Add new builder tools tag
  • Add chain-index tag to Oura project

@fill-the-fill fill-the-fill added the builder tool Indicates a PR/issue on a builder tool label Apr 4, 2022
katomm
katomm requested changes Apr 5, 2022
View reviewed changes
Copy link
Member

@katomm katomm left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Chain index is a good tag to have. I know we lack of a tag guideline but please let's stick to only lowercase tags and from my feeling the dash in the middle of the label should go away:

  • "chainindex" for the tag.
  • "Chain Index" for the label.

(open for discussions of course)

@katomm katomm merged commit 513a5dc into cardano-foundation:staging Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rdlrt rdlrt rdlrt approved these changes

@katomm katomm katomm approved these changes

Assignees
No one assigned
Labels
builder tool Indicates a PR/issue on a builder tool
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@fill-the-fill @rdlrt @katomm @fickevics-makor