Skip to content

Make the Linter Yell At Us: Create lint.yml - testing lint options#26

Open
lwasser wants to merge 3 commits into
mainfrom
linting-2
Open

Make the Linter Yell At Us: Create lint.yml - testing lint options#26
lwasser wants to merge 3 commits into
mainfrom
linting-2

Conversation

@lwasser

@lwasser lwasser commented Dec 16, 2020

Copy link
Copy Markdown

No description provided.

@lwasser lwasser changed the title Create lint.yml - testing lint options Make the Linter Yell At Us: Create lint.yml - testing lint options Dec 16, 2020

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Lintly has detected code quality issues in this pull request.

Comment thread agu_oss/clean_data.py

temp_df = pd.read_csv(url,
skiprows=69,
skiprows = 69,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

E251: unexpected spaces around keyword / parameter equals

Comment thread agu_oss/clean_data.py

temp_df = pd.read_csv(url,
skiprows=69,
skiprows = 69,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

E251: unexpected spaces around keyword / parameter equals

Comment thread agu_oss/clean_data.py
"""A second sample module to clean data."""

import pandas as pd
import numpy as np

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

F401: 'numpy as np' imported but unused

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