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

CR exit criteria #1736

Merged
merged 4 commits into from Nov 2, 2022
Merged

CR exit criteria #1736

merged 4 commits into from Nov 2, 2022

Conversation

mmccool
Copy link
Contributor

@mmccool mmccool commented Nov 1, 2022

  • Add paragraph specifying CR exit criteria to sotd section in publication/ver11/5-cr
  • List ALL statements at risk in sotd (overlooked some Thing Model ones, and added S&P ones)
  • Updated IR template - tooltips
  • Rerun index.html -(ReSpec export) -> static.html -(htmltidy)-> Overview.html processing

I also rebased before running this and seem to have picked up some edits to the change log. Please check that the result is correct, it looks like just formatting changes. Note that "static.html" and "Overview.html" get overwritten by the above processing so any edits should be made to "index.html". The old version did not have the links filled in which should have happened with the ReSpec export above so I think "static.html" might have been mistakenly edited.

Update: Sebastian has checked and the change log looks to be ok.

@mmccool
Copy link
Contributor Author

mmccool commented Nov 1, 2022

Sorry, just added a few more things to this PR. Apologize for lumping somewhat unrelated things together, but I don't think any of these are controversial and I wanted to avoid merge conflicts:

  • Listed ALL at-risk assertions in sotd section (this is another CR transition requirement; if we leave any out, it may delay the transition)
  • Updated IR report template to add tooltips to table headers

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.

None yet

2 participants