search

zephyrproject-rtos / reqmgmt

8 stars 10 forks source link

requirements: updated Interrupts sections #61

Closed Timotheous closed 1 month ago

Timotheous commented 3 months ago 
          updated system_requirements Interrupts section
          updated software_requirements Interrupts section
simhein commented 2 months ago

@Timotheous Sorry for my really late re-review on that. It looks very good now. Please rebase this PR aswell

simhein commented 2 months ago

LGTM now

@Timotheous Sorry I was a little bit fast with the approval. There is a CI error due to the not found Requirement UIDs in the linkage This needs to be fixed to make the CI pass.

Timotheous commented 1 month ago

My fault. I failed to run strictdoc and build the HTML.

Updated the offending links. Strictdoc now runs locally. Squashed and force pushed to PR.

Seems correct now. Except for one question above.

Timotheous commented 1 month ago

Not sure of the issue with .gitignore. I removed all the lines that I added.

simhein commented 1 month ago

Not sure of the issue with .gitignore. I removed all the lines that I added.

True that was just a odd view from the github review diff view. You can ignore that comment

Timotheous commented 1 month ago

Change made. Should be good now.