Skip to content

Fix mag value error - #1908

Merged
maxineofficial merged 1 commit into
IMAP-Science-Operations-Center:devfrom
maxineofficial:fix-mag-value-error
Jul 3, 2025
Merged

Fix mag value error#1908
maxineofficial merged 1 commit into
IMAP-Science-Operations-Center:devfrom
maxineofficial:fix-mag-value-error

Conversation

@maxineofficial

Copy link
Copy Markdown
Collaborator

Change Summary

Overview

New Dependencies

New Files

  • new file 1
    • description of new file 1's purpose

Deleted Files

  • deleted file 1
    • explanation for why file was deleted

Updated Files

  • updated file 1
    • description of change 1 in file 1
    • description of change 2 in file 2
  • updated file 2
    • descipriton of change 1 in file 2

Testing

@maxineofficial
maxineofficial merged commit 608d4b1 into IMAP-Science-Operations-Center:dev Jul 3, 2025
10 of 11 checks passed
@greglucas

Copy link
Copy Markdown
Collaborator

This did not wait for any tests to pass or approvals. I have concerns with this approach. Was there a reason this needed to be done before anyone else could review / wait for automated tests? I don't think this rises to the level of immediate hot-fix.

@maxineofficial

Copy link
Copy Markdown
Collaborator Author

@greglucas definitely agree, this was actually an accident. I meant to merge this into my fork dev branch, but didn't notice that the PR was actually for the real dev branch. I'll revert once the end-to-end test is done and redo it properly.

@bourque bourque added this to IMAP Jul 31, 2025
@bourque bourque added this to the July 2025 milestone Jul 31, 2025
@bourque bourque moved this to Done in IMAP Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants