Skip to content

Comments

Pawel plesniak/remove xml#777

Merged
PawelPlesniak merged 2 commits intodevelopfrom
PawelPlesniak/RemoveXML
Feb 16, 2026
Merged

Pawel plesniak/remove xml#777
PawelPlesniak merged 2 commits intodevelopfrom
PawelPlesniak/RemoveXML

Conversation

@PawelPlesniak
Copy link
Collaborator

@PawelPlesniak PawelPlesniak commented Feb 16, 2026

Description

There was a change of configuration extension from e.g. run_conf1.data.xml to run_conf1.xml.data. This was not announced and is not accounted for in the gitignores, as such mistakenly got merged in. This PR deletes the file and corrects the gitignore.

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. python -m pytest)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future.
    (Indicate issue here: # (issue))

@PawelPlesniak
Copy link
Collaborator Author

MSQT run and passed.

@emmuhamm emmuhamm self-requested a review February 16, 2026 14:51
Copy link
Contributor

@emmuhamm emmuhamm left a comment

Choose a reason for hiding this comment

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

LGTM thumbsup

@PawelPlesniak PawelPlesniak merged commit bd2cfe7 into develop Feb 16, 2026
3 of 4 checks passed
@PawelPlesniak PawelPlesniak deleted the PawelPlesniak/RemoveXML branch February 16, 2026 14:51
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.

2 participants