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

Update issue templates #3019

Merged
merged 1 commit into from
Jun 15, 2019
Merged

Update issue templates #3019

merged 1 commit into from
Jun 15, 2019

Conversation

max-sixty
Copy link
Collaborator

@max-sixty max-sixty commented Jun 12, 2019

This:

  • Updates to the newer GitHub format
  • Strengthens the language around MCVE. It doesn't say people have to, but makes it less of a suggestion
  • Only contains 'Bug Report'. Should we have others? I don't immediately see how they'd be different

(I realize because I did this from the GH website, it created from this repo rather than my own fork. That's a mistake.)

@max-sixty
Copy link
Collaborator Author

Some ideation re MCVE:

An app that would be generally useful: an easy way for people to post an runnable example. e.g. Rust has Rust Playground. That would force people to have an example that runs; no risk of copy-paste errors.

Has anyone tried with a Colab link?

Stack Overflow would have the scale more than xarray-on-GH, so maybe we're not the ones to forge new ground...

@shoyer
Copy link
Member

shoyer commented Jun 15, 2019

Has anyone tried with a Colab link?

I link this idea, we'd just have to ask users to update xarray first, since Colab updates xarray on their own schedule.

But let's go ahead and merge this for now anyways.

@shoyer shoyer merged commit c2a2a6e into master Jun 15, 2019
@shoyer shoyer deleted the max-sixty-patch-1 branch June 15, 2019 03:35
dcherian added a commit to dcherian/xarray that referenced this pull request Jun 24, 2019
* master: (31 commits)
  Add quantile method to GroupBy (pydata#2828)
  rolling_exp (nee ewm) (pydata#2650)
  Ensure explicitly indexed arrays are preserved (pydata#3027)
  add back dask-dev tests (pydata#3025)
  ENH: keepdims=True for xarray reductions (pydata#3033)
  Revert cmap fix (pydata#3038)
  Add "errors" keyword argument to drop() and drop_dims() (pydata#2994) (pydata#3028)
  More consistency checks (pydata#2859)
  Check types in travis (pydata#3024)
  Update issue templates (pydata#3019)
  Add pytest markers to avoid warnings (pydata#3023)
  Feature/merge errormsg (pydata#2971)
  More support for missing_value. (pydata#2973)
  Use flake8 rather than pycodestyle (pydata#3010)
  Pandas labels deprecation (pydata#3016)
  Pytest capture uses match, not message (pydata#3011)
  dask-dev tests to allowed failures in travis (pydata#3014)
  Fix 'to_masked_array' computing dask arrays twice (pydata#3006)
  str accessor (pydata#2991)
  fix safe_cast_to_index (pydata#3001)
  ...
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.

2 participants