Skip to content

Conversation

@cameel
Copy link
Collaborator

@cameel cameel commented Dec 15, 2021

argotorg/solidity-blog#159 (comment) by @christianparpart gave me and idea that we could automatically apply labels in our issue templates.

The PR also does some minor cleanup in templates:

  • Replaces tabs with spaces
  • Removes hard wrapping to make the instructions look more concise. Text in Github's issue editor is always soft wrapped. It also will never show up in the preview and even if it did, Github does not remove hard wrapping when rendering issue comments as would normally happen in markdown.

- Support questions are better asked in one of the following locations:
- [Solidity chat](https://gitter.im/ethereum/solidity)
- [Stack Overflow](https://ethereum.stackexchange.com/)
- [Solidity chat](https://gitter.im/ethereum/solidity)
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want to reference the linked Matrix chat here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Personally, I always give people the gitter link because it just drops you directly on the channel without having to go through any nag screens about downloading the client.

Maybe we should list both?

Copy link
Contributor

Choose a reason for hiding this comment

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

That is true, so let's stay with gitter as long as it still works.

---
name: Bug Report
about: Bug reports about the Solidity Compiler.
labels: ["bug :bug:"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Currently (at least that's how I have handled it), the label served the purpose of "confirmed bug". What do we want to do about that? Just keep the label and use the project columns to track confirmed / not confirmed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, I had doubts about this myself because of that system. Maybe we should actually change it? I think it would be useful to have this stuff immediately categorized and a confirmed label would have more obvious meaning - the current system is a bit too implicit.

But I can remove it from the template if you prefer to stay with the current system.

Copy link
Contributor

Choose a reason for hiding this comment

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

let's use the columns in the project to mark it as confirmed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ok, so not in New issues means that it's confirmed?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes

@cameel cameel merged commit 76e83a9 into develop Dec 16, 2021
@cameel cameel deleted the labels-in-issue-templates branch December 16, 2021 16:40
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.

4 participants