Skip to content

added pr and issues template#499

Merged
Plamen5kov merged 2 commits intomasterfrom
plamen5kov/add_issue_pr_template
Jun 30, 2016
Merged

added pr and issues template#499
Plamen5kov merged 2 commits intomasterfrom
plamen5kov/add_issue_pr_template

Conversation

@Plamen5kov
Copy link
Copy Markdown
Contributor

@Plamen5kov Plamen5kov commented Jun 24, 2016

In the android runtime there are a lot of issues which could be better described, and this PR will help people, opening issues in the android runtime, provide all the necessary information for the issue resolution. Same goes of PR.

  • adds an issue template
  • adds a PR template

Most of the content in these files is from https://github.com/NativeScript/nativescript, with a couple of adjustments, specific for the android runtime.

@ping @NativeScript/android-runtime

@ns-bot
Copy link
Copy Markdown

ns-bot commented Jun 24, 2016

💚

Comment thread .github/ISSUE_TEMPLATE.md
Please, ensure your title is less than 63 characters long and starts with a capital
letter.

### Please provide the following version numbers that your issue occurs with:
Copy link
Copy Markdown
Contributor

@petekanev petekanev Jun 24, 2016

Choose a reason for hiding this comment

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

If the issue is regarding a crash during build, we could also ask users to execute tns <command> --log trace > cli-log.txt and attach it to the Issue

If the issue is regarding a crash during execution, we may ask them to run write __enableVerboseLogging() in the beginning of their app.js/ts, then execute tns <run/...> android > logcat.txt and also attach it to the open issue.
This will potentially save us time in communicating back and forth asking for logs

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

will do

@Plamen5kov Plamen5kov force-pushed the plamen5kov/add_issue_pr_template branch from 37bec56 to e8e72d6 Compare June 24, 2016 12:42
Comment thread .github/ISSUE_TEMPLATE.md
<!--
Please provide us with the necessary information to reproduce your problem. That includes but is not limited to presenting us with steps to reproduce the problem, or a small repo example.
-->

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add a Workaround/ Possible fix paragraph?

Copy link
Copy Markdown
Contributor Author

@Plamen5kov Plamen5kov Jun 24, 2016

Choose a reason for hiding this comment

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

I think this part better stay in the comments, but if we decide otherwise, I will add such a section.

@ns-bot
Copy link
Copy Markdown

ns-bot commented Jun 24, 2016

💚

2 similar comments
@ns-bot
Copy link
Copy Markdown

ns-bot commented Jun 24, 2016

💚

@ns-bot
Copy link
Copy Markdown

ns-bot commented Jun 24, 2016

💚

@Plamen5kov Plamen5kov merged commit 590ff79 into master Jun 30, 2016
@Plamen5kov Plamen5kov deleted the plamen5kov/add_issue_pr_template branch July 4, 2016 10:34
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.

3 participants