Skip to content

Adds header to requirements.txt for s3-based deploy option#601

Merged
elijahbenizzy merged 1 commit intomainfrom
elijah/additional-licenses
Nov 29, 2025
Merged

Adds header to requirements.txt for s3-based deploy option#601
elijahbenizzy merged 1 commit intomainfrom
elijah/additional-licenses

Conversation

@elijahbenizzy
Copy link
Contributor

Once we looked at the wheel these were missing

Changes

  • license to a requirements.txt file
  • license to the README
  • license to .asf.yaml

How I tested this

Notes

Checklist

  • PR has an informative and human-readable title (this will be pulled into the release notes)
  • Changes are limited to a single goal (no scope creep)
  • Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future TODOs are captured in comments
  • Project documentation has been updated if adding/changing functionality.

@elijahbenizzy elijahbenizzy force-pushed the elijah/additional-licenses branch from 4cb16fd to 6e8411a Compare November 29, 2025 03:13
@elijahbenizzy elijahbenizzy marked this pull request as ready for review November 29, 2025 03:13
@elijahbenizzy elijahbenizzy enabled auto-merge (squash) November 29, 2025 03:13
@github-actions
Copy link

github-actions bot commented Nov 29, 2025

A preview of 6e8411a is uploaded and can be seen here:

https://burr.dagworks.io/pull/601

Changes may take a few minutes to propagate. Since this is a preview of production, content with draft: true will not be rendered. The source is here: https://github.com/apache/burr/tree/gh-pages/pull/601/

@elijahbenizzy elijahbenizzy merged commit 8168535 into main Nov 29, 2025
11 checks passed
@elijahbenizzy elijahbenizzy deleted the elijah/additional-licenses branch November 29, 2025 05:12
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