Skip to content

Conversation

@rahulr-NI
Copy link
Contributor

@rahulr-NI rahulr-NI commented Apr 9, 2025

- [ ] I've added tests applicable for this pull request

What does this Pull Request accomplish?

Since we are supporting version bump for each drivers, the CHANGELOG.md was modified for each driver.
Now each driver will be having separate section within the CHANGELOG.md.
Driver lists are alphabetically arranged.
The versions under each driver are arrange in such a way that latest version comes on top.
There are two index. One for drivers. Other within each driver for releases.
Initial Release of a driver will be the first version number for all drivers.

List issues fixed by this Pull Request below, if any.

What testing has been done?

Validated whether the drivers are in alphabetical order.
Validated the version present for each drivers arrange chronologically.
There is an index for drivers and each driver has other index for version numbers.
Initial version of the driver is the first release for that driver.
Same indexation is maintained as present in original changelog.md file.

@codecov-commenter
Copy link

codecov-commenter commented Apr 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.26%. Comparing base (96b2711) to head (2d3f469).

❗ There is a different number of reports uploaded between BASE (96b2711) and HEAD (2d3f469). Click for more details.

HEAD has 11 uploads less than BASE
Flag BASE (96b2711) HEAD (2d3f469)
nifakeunittests 5 4
nidcpowerunittests 5 4
nimodinstsystemtests 1 0
nisesystemtests 1 0
niswitchsystemtests 1 0
nidmmsystemtests 1 0
niscopesystemtests 1 0
nifgensystemtests 1 0
nidcpowersystemtests 1 0
nitclksystemtests 1 0
nidigitalsystemtests 1 0
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2091      +/-   ##
==========================================
- Coverage   91.34%   82.26%   -9.08%     
==========================================
  Files          66       28      -38     
  Lines       16292     4004   -12288     
==========================================
- Hits        14882     3294   -11588     
+ Misses       1410      710     -700     
Flag Coverage Δ
codegenunittests 84.95% <ø> (ø)
nidcpowersystemtests ?
nidcpowerunittests 89.53% <ø> (ø)
nidigitalsystemtests ?
nidigitalunittests 68.44% <ø> (ø)
nidmmsystemtests ?
nifakeunittests 86.98% <ø> (-0.26%) ⬇️
nifgensystemtests ?
nimodinstsystemtests ?
nimodinstunittests 94.20% <ø> (ø)
niscopesystemtests ?
niscopeunittests 43.20% <ø> (ø)
nisesystemtests ?
niswitchsystemtests ?
nitclksystemtests ?
nitclkunittests 98.26% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 43 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 96b2711...2d3f469. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ni-jfitzger
Copy link
Collaborator

This PR includes unrelated build_release.py tooling changes and the associated changes to release process documentation.

@rahulr-NI
Copy link
Contributor Author

This PR includes unrelated build_release.py tooling changes and the associated changes to release process documentation.

Hi Jay..,, I have updated the files.

@ni-jfitzger
Copy link
Collaborator

I've fully reviewed the NI-DCPower section. Still need to review the rest.

I've now reviewed the NI-Digital section, as well.

NI-DMM section reviewed.

NI-FGEN section reviewed.

NI-ModInst section reviewed.

NI-SCOPE section reviewed.

@ni-jfitzger
Copy link
Collaborator

I've fully reviewed the NI-DCPower section. Still need to review the rest.

I've now reviewed the NI-Digital section, as well.

NI-DMM section reviewed.

NI-FGEN section reviewed.

NI-ModInst section reviewed.

NI-SCOPE section reviewed.

NI Switch Executive, NI-SWITCH and NI-TClk sections reviewed. That's all sections.

Copy link
Collaborator

@ni-jfitzger ni-jfitzger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert any changes to CONTRIBUTING.md meant to reflect tooling changes. The only changes to the document should be those related to the CHANGELOG.

@rahulr-NI
Copy link
Contributor Author

Please revert any changes to CONTRIBUTING.md meant to reflect tooling changes. The only changes to the document should be those related to the CHANGELOG.

Updated

@ni-jfitzger ni-jfitzger merged commit 6f386e4 into ni:master Apr 22, 2025
9 of 10 checks passed
@rahulr-NI rahulr-NI deleted the users/rahur/changelog_for_individual_drivers branch July 10, 2025 06:54
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.

Update CHANGELOG system for independent package versioning

4 participants