Skip to content

Conversation

paul-paliychuk
Copy link
Contributor

@paul-paliychuk paul-paliychuk commented Oct 10, 2024

Important

Add Apache License 2.0 to the project and update .fernignore to include it.

  • License:
    • Added LICENSE file with Apache License 2.0 terms and conditions.
    • Updated .fernignore to include LICENSE file.

This description was created by Ellipsis for 4336d11. It will automatically update as commits are pushed.

@paul-paliychuk paul-paliychuk merged commit 4eb86c4 into main Oct 10, 2024
3 checks passed
@paul-paliychuk paul-paliychuk deleted the add-license branch October 10, 2024 22:01
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 4336d11 in 7 seconds

More details
  • Looked at 219 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 drafted comments based on config settings.
1. .fernignore:12
  • Draft comment:
    Add a newline at the end of the file for consistency and to adhere to best practices.
  • Reason this comment was not posted:
    Confidence changes required: 10%
    The .fernignore file is missing a newline at the end, which is a common best practice for text files.
2. LICENSE:201
  • Draft comment:
    Add a newline at the end of the file for consistency and to adhere to best practices.
  • Reason this comment was not posted:
    Confidence changes required: 10%
    The LICENSE file is missing a newline at the end, which is a common best practice for text files.

Workflow ID: wflow_LlWWhonVFGyo7NrD


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

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.

1 participant