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

data batch docstring re-formatting: 'Examples:' -> 'Examples' #362

Conversation

sadielbartholomew
Copy link
Member

@sadielbartholomew sadielbartholomew commented Mar 22, 2022

As for #182 over on cfdm, but note that this is only across the data module, i.e:

$ pwd
<path to cf repo>/cf-python/cf/data
$ find . -type f | xargs sed -i 's/Examples:/Examples/g'

in line with our approach to managing development of mostly data modules on lama-to-dask and all other modules on master. I am about to open a corresponding PR on master to cover all non-data modules.

@davidhassell I have opened this PR so the batch update gets done at the first available point, but please prioritise merging of your currently-open PRs before this - it is trivial to manage code updates and to resolve merge conflicts here, much less so (probably) on your PRs.

@sadielbartholomew sadielbartholomew added documentation Improvements or additions to documentation code tidy dask Relating to the use of Dask labels Mar 22, 2022
@sadielbartholomew sadielbartholomew self-assigned this Mar 22, 2022
@davidhassell
Copy link
Collaborator

Great. Can we just merge this now and I'll check for conflicts in the PRs that are currently up?

@sadielbartholomew
Copy link
Member Author

Great. Can we just merge this now and I'll check for conflicts in the PRs that are currently up?

Sure, whatever you prefer - I just didn't want to create merge difficulties for you, though git may handle it neatly (I suspect it will but I'm not certain)!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code tidy dask Relating to the use of Dask documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants