Skip to content

JCF: update the code to properly reflect the conditions of the upstre…#33

Merged
jcfreeman2 merged 2 commits into
developfrom
johnfreeman/license_remediation
Jun 22, 2026
Merged

JCF: update the code to properly reflect the conditions of the upstre…#33
jcfreeman2 merged 2 commits into
developfrom
johnfreeman/license_remediation

Conversation

@jcfreeman2

Copy link
Copy Markdown
Collaborator

…am package's Apache 2.0 license

Description

This PR brings the code into compliance with the Apache 2.0 license of the upstream package (ATLAS's "config"). To test, convince yourself that the code is, in fact, now in compliance.

Type of change

  • [X ] Documentation (non-breaking change that adds or improves the documentation)
  • New feature or enhancement (non-breaking change which adds functionality)
  • Optimization (non-breaking change that improves code/performance)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Testing checklist

  • Unit tests pass (e.g. dbt-build --unittest)
  • Minimal system quicktest passes (pytest -s minimal_system_quick_test.py)
  • Full set of integration tests pass (dunedaq_integtest_bundle.sh)
  • Python tests pass if applicable (e.g. python -m pytest)
  • Pre-commit hooks run successfully if applicable (e.g. pre-commit run --all-files)

Comments here on the testing

Further checks

  • Code is commented where needed, particularly in hard-to-understand areas
  • Code style is correct (dbt-build --lint, and/or see https://dune-daq-sw.readthedocs.io/en/latest/packages/styleguide/)
  • If applicable, new tests have been added or an issue has been opened to tackle that in the future.
    (Indicate issue here: # (issue))

@jcfreeman2 jcfreeman2 merged commit cf6a9a4 into develop Jun 22, 2026
@jcfreeman2 jcfreeman2 deleted the johnfreeman/license_remediation branch June 22, 2026 23: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.

3 participants