Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add headers and lint #72

Merged
merged 3 commits into from
Nov 18, 2022
Merged

Add headers and lint #72

merged 3 commits into from
Nov 18, 2022

Conversation

t-young31
Copy link
Member

Resolves #68

@t-young31 t-young31 requested review from nels and AnikaC-git November 18, 2022 11:12
@t-young31 t-young31 mentioned this pull request Nov 18, 2022
Merged
AnikaC-git
AnikaC-git previously approved these changes Nov 18, 2022
View reviewed changes
Copy link
Contributor

@AnikaC-git AnikaC-git left a comment

Choose a reason for hiding this comment

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

If @nels is happy that UCL is not included, so am I! 😸

cli/src/pixl_cli/__init__.py Outdated Show resolved Hide resolved
cli/src/pixl_cli/__init__.py Outdated Show resolved Hide resolved
@t-young31 t-young31 requested a review from AnikaC-git November 18, 2022 15:10
nels
nels approved these changes Nov 18, 2022
View reviewed changes
Copy link
Contributor

@nels nels left a comment

Choose a reason for hiding this comment

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

Thank you 👍

@nels nels merged commit 410723c into main Nov 18, 2022
@nels nels deleted the t-young/add_headers branch November 18, 2022 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nels nels nels approved these changes

@AnikaC-git AnikaC-git Awaiting requested review from AnikaC-git

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add headers and CI check
3 participants
@t-young31 @nels @AnikaC-git