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.

Sign up for GitHub

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

Jump to bottom

resolve TODO & FIXME markers #272

Merged
merged 12 commits into from
May 24, 2022
Merged

resolve TODO & FIXME markers #272

merged 12 commits into from
May 24, 2022

Conversation

prjemian
Copy link
Owner

@prjemian prjemian commented May 5, 2022

@prjemian prjemian added this to the 2021.2.1 milestone May 5, 2022
@prjemian prjemian self-assigned this May 5, 2022
@coveralls
Copy link

coveralls commented May 5, 2022 •
edited
Loading

Coverage Status

Coverage decreased (-0.009%) to 95.722% when pulling 56d8e40 on 265-resolve-TODO-and-FIXME into 6a794b8 on main.

@prjemian
Copy link
Owner Author

prjemian commented May 5, 2022

The Codacy suggestion is now an issue (#273) for another release

@prjemian prjemian merged commit cbd1ca0 into main May 24, 2022
@prjemian prjemian deleted the 265-resolve-TODO-and-FIXME branch May 24, 2022 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@prjemian prjemian

Labels
documentation task
Projects
None yet
Milestone
2021.2.1
Development

Successfully merging this pull request may close these issues.

Resolve existing FIXME & TODO items in NeXus file docs
2 participants
@prjemian @coveralls