Skip to content

Conversation

@ehanson8
Copy link
Contributor

Purpose and background context

Updates app according to our maintenance week documentation.

How can a reviewer manually see the effects of these changes?

Run make test and make lint to confirm they still pass

Includes new or updated dependencies?

YES

Changes expectations for external applications?

NO

What are the relevant tickets?

NA

Code review

  • Code review best practices are documented here and you are encouraged to have a constructive dialogue with your reviewers about their preferences and expectations.

* Update dependencies
* Remove unused ruff ignore
* Add CODEOWNERS file
* Update PR template
@ehanson8 ehanson8 requested a review from a team November 25, 2025 18:30
@ghukill ghukill self-assigned this Nov 26, 2025
Copy link
Contributor

@ghukill ghukill left a comment

Choose a reason for hiding this comment

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

Was able to run the Dev1 tests from the README, pointing at this file in S3: s3://ghukill-test/bursar/input/test-1234.xml.

Approved!

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