-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[Docs] CONTRIBUTING.md Reporting A Bug outdated #18460
Comments
I agree that these seem like the best things folks can do at the moment, since twiddle doesn’t work with the most recent versions of Ember |
I'd personally prefer to suggest "making a demo repo on GitHub". I wish CodeSandbox was a good enough suggestion, but I just don't find it stable enough to be able to use it to quickly dig into a reported issue. |
I’ve been adding |
You've raised a very good issue, @Alonski ! Currently, Contributing docs mention 2 tools:
To cover the whole range of possible issues, it would be ideal if a reproduction environment:
Unfortunately, it seems there is no ideal solution right now. The following online IDEs could be a good solutions in the future:
From my point of view, while there is no reliable online IDE, ember-twiddle is still a very good option for reproducing simple issues quickly. For more complex scenarios, the docs could recommend a GitHub repository with an Ember CLI project. However, as @mike-north mentioned, ember-twiddle requires much development/maintenance effort, and it is specific for Ember.js. Therefore, it is likely worth the price to eventually migrate to one of the popular multipurpose online IDEs. Thank you, @mike-north, for all your awesome job on this! |
Closed by #18930 |
Here: https://github.com/emberjs/ember.js/blob/master/CONTRIBUTING.md#reporting-a-bug
Number 3: States that a Twiddle or JSBin should be provided to showcase the bug.
This should be updated as those aren't necessarily the best ways to showcase a bug.
Possible solutions:
The text was updated successfully, but these errors were encountered: