Skip to content

Conversation

@elasticspoon
Copy link
Collaborator

@elasticspoon elasticspoon commented Sep 10, 2025

Adds shopify rubocop with various applicable plugins

Some stuff needs fixing but for now I added inline todos for those

@elasticspoon elasticspoon marked this pull request as ready for review September 10, 2025 23:01
@elasticspoon elasticspoon changed the title add lint feat: add shopify rubocop Sep 10, 2025

def project_workshop_logs(date, windows_type, project_id)
if project_id
if project_id
Copy link
Collaborator

Choose a reason for hiding this comment

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

not sure why this got indented, but ok

Copy link
Collaborator

Choose a reason for hiding this comment

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

one space to two. 😆

@github-actions
Copy link

This pull request has been inactive for 30 days. Please update it or let us know if it is still relevant.

1 similar comment
@github-actions
Copy link

This pull request has been inactive for 30 days. Please update it or let us know if it is still relevant.

@elasticspoon elasticspoon deleted the add-lint branch November 17, 2025 06:16
@maebeale
Copy link
Collaborator

maebeale commented Nov 17, 2025

@elasticspoon what was wrong w this PR? (asking bc we want a linter to complete the CI updates)

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.

4 participants