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

chore: refactor issue template with forum struct (yaml format) #1531

Merged
merged 2 commits into from
Jul 1, 2021

Conversation

imbajin
Copy link
Member

@imbajin imbajin commented Jul 1, 2021

After use it, submit issue will be more user friendly, main diff here:

  1. the whole UI experience is much more better (a string of enhancement)
  2. users don't need to update / write markdown by themselves
  3. we could define the highlight code block by default language
  4. description will only show before issue submitted, no need to hide them by any trick way.. (such like <details> or html-comment)
  5. we can choose required info now (otherwise cannot submit issue)

PS: also reduce the daily stale issues amount from 25 -> 10

the UI before submit like: (click here to preview)
image

after submit like:

image

After use it, submit issue will be more user friendly, main diff here:
1. the whole UI experience is much more better (a string of enhancement)
2. users don't need to update / write markdown by themselves
3. we could define the highlight code block by default language
4. description will only show before issue submitted, no need to hide them by any trick way.. (such like `details` or html-comment `<!-- -->`)
5. we can choose required info now (otherwise cannot submit issue)

PS: also reduce the daily stale issues amount from 25 -> 10
@codecov
Copy link

codecov bot commented Jul 1, 2021

Codecov Report

Merging #1531 (615fbd4) into master (23beacd) will increase coverage by 14.88%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master    #1531       +/-   ##
=============================================
+ Coverage     44.77%   59.66%   +14.88%     
- Complexity     4619     6152     +1533     
=============================================
  Files           410      410               
  Lines         33272    33272               
  Branches       4590     4590               
=============================================
+ Hits          14899    19851     +4952     
+ Misses        16633    11349     -5284     
- Partials       1740     2072      +332     
Impacted Files Coverage Δ
...du/hugegraph/backend/tx/GraphIndexTransaction.java 81.04% <0.00%> (+0.23%) ⬆️
...m/baidu/hugegraph/backend/tx/GraphTransaction.java 79.84% <0.00%> (+0.32%) ⬆️
...ava/com/baidu/hugegraph/structure/HugeElement.java 70.77% <0.00%> (+0.45%) ⬆️
...n/java/com/baidu/hugegraph/structure/HugeEdge.java 76.49% <0.00%> (+0.46%) ⬆️
...om/baidu/hugegraph/task/StandardTaskScheduler.java 75.60% <0.00%> (+0.48%) ⬆️
.../baidu/hugegraph/backend/tx/SchemaTransaction.java 91.50% <0.00%> (+0.50%) ⬆️
...u/hugegraph/schema/builder/PropertyKeyBuilder.java 88.81% <0.00%> (+0.62%) ⬆️
...aidu/hugegraph/backend/tx/AbstractTransaction.java 70.06% <0.00%> (+0.63%) ⬆️
.../java/com/baidu/hugegraph/structure/HugeIndex.java 80.67% <0.00%> (+0.84%) ⬆️
...om/baidu/hugegraph/backend/query/QueryResults.java 71.15% <0.00%> (+0.96%) ⬆️
... and 187 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23beacd...615fbd4. Read the comment docs.

@imbajin imbajin changed the title chore: replace issue template with forum struct (format by yaml) chore: refactor issue template with forum struct (yaml format) Jul 1, 2021
zhoney
zhoney previously approved these changes Jul 1, 2021
Linary
Linary previously approved these changes Jul 1, 2021
@imbajin imbajin dismissed stale reviews from Linary and zhoney via 615fbd4 July 1, 2021 10:33
@javeme javeme merged commit 4cb4348 into master Jul 1, 2021
@javeme javeme deleted the issue-forum-template branch July 1, 2021 11:24
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.

None yet

4 participants