Skip to content

Update release notes for PRs 60 and 74#75

Merged
keenanlang merged 2 commits intoepics-modules:masterfrom
simon-ess:release_notes
Jun 12, 2025
Merged

Update release notes for PRs 60 and 74#75
keenanlang merged 2 commits intoepics-modules:masterfrom
simon-ess:release_notes

Conversation

@simon-ess
Copy link
Collaborator

Release notes for the changes in #60 and #74 are added here. In particular, it is highlighted that #74 adds a breaking change.

@keenanlang keenanlang merged commit 16bbdb7 into epics-modules:master Jun 12, 2025
0 of 4 checks passed
@mdavidsaver
Copy link
Contributor

It looks like this PR was merged while still failing CI due to the addition of an undefined variable status.

   ../save_restore.c:2095:9: error: use of undeclared identifier 'status'
          status = write_save_file(plist, NULL, NULL);
          ^
  ../save_restore.c:2096:13: error: use of undeclared identifier 'status'; did you mean 'stat'?
          if (status != 0) {
              ^~~~~~
              stat

@mdavidsaver
Copy link
Contributor

Ah, wrong PR. #70 is the culprit.

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.

3 participants