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.

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

Add travis-lint #35

Closed
wants to merge 3 commits into from
Closed

Add travis-lint #35

wants to merge 3 commits into from

Conversation

stephenfin
Copy link
Contributor

Contains the Travis-related contents of #29.

stephenfin and others added 2 commits June 2, 2020 11:01
Trivial changes highlighted by my editor. We can/should configure
pre-commit to do this in the future.

Change-Id: Ib62be1ce61bb73702db8b31fc1a9b1fb8e806a36
Signed-off-by: Stephen Finucane <stephen@that.guru>
- exclude running jobs on tags
- perform linting of travis file
- remove credentials, easier to manage via web

Change-Id: Iafa4a5ef0404b18fa8b2dcb1553d5f9aaed6c92b
@ssbarnea
Copy link
Member

ssbarnea commented Jun 2, 2020

@stephenfin should we get in sync on irc about addressing chores on doc8? My nick is "zbr" there.

@stephenfin
Copy link
Contributor Author

@stephenfin should we get in sync on irc about addressing chores on doc8? My nick is "zbr" there.

Yup. stephenfin there too.

Talking about this PR specifically, it seems the gem installation still isn't functioning correctly. I simply rebased the change and stripped out the README rework so I'm not sure where to start with this. Care to take a look?

@ssbarnea ssbarnea added the skip-changelog Can be missed from the changelog. label Aug 15, 2020
@ssbarnea
Copy link
Member

Closing it as no longer needed once we switch to github actions, faster, more reliable and easier to maintain. I am almost done with migration.

@ssbarnea ssbarnea closed this Aug 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Can be missed from the changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants