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

(v0.5 backport) lint: backport bitHoud-related commits #120

Closed
wants to merge 7 commits into from

Conversation

aqrln
Copy link
Member

@aqrln aqrln commented Apr 3, 2017

This PR fixes failing bitHoud checks for v0.5.

This is a partial backport of 4f83d92
and a backport of 60ae973 together with
that.
@aqrln aqrln added the test label Apr 3, 2017
@aqrln aqrln requested a review from belochub April 3, 2017 12:27
@aqrln aqrln changed the title (v0.5 backport) lint: add bitHound config (v0.5 backport) lint: backport bitHoud-related commits Apr 3, 2017
This is kind of backports PR #105 in a different way, and this way
might be even better than that the original commit used.

The reason of doing this for v0.5, besides fixing a bug, is the fact
that we cannot use metasync in v0.5 since it is not compatible with old
Node.js versions while old library versions that used to be are too
outdated and even get reported by bitHound as insecure.

Backport-of: #105
@aqrln
Copy link
Member Author

aqrln commented Apr 3, 2017

@belochub bitHoud is now happy, can you please take a look at this PR?

@belochub
Copy link
Member

belochub commented Apr 3, 2017

CI fails with some errors during installation. Can you check this out, please?

@aqrln
Copy link
Member Author

aqrln commented Apr 3, 2017

Ugh... messed up with resolving conflicts in cc78133.

Update the Markdown linting tools.

Backport-of: #87
Copy link
Member

@belochub belochub left a comment

Choose a reason for hiding this comment

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

LGTM

aqrln added a commit that referenced this pull request Apr 3, 2017
This is a partial backport of 4f83d92
and a backport of 60ae973 together with
that.

PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
aqrln added a commit that referenced this pull request Apr 3, 2017
This is a backport of eb49729.

PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
aqrln added a commit that referenced this pull request Apr 3, 2017
This is kind of backports PR #105 in a different way, and this way
might be even better than that the original commit used.

The reason of doing this for v0.5, besides fixing a bug, is the fact
that we cannot use metasync in v0.5 since it is not compatible with old
Node.js versions while old library versions that used to be are too
outdated and even get reported by bitHound as insecure.

Backport-of: #105
PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
aqrln added a commit that referenced this pull request Apr 3, 2017
Original commit: 16d8c85

PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
aqrln added a commit that referenced this pull request Apr 3, 2017
This is a backport of eafafd1,
6069f18 and
691795b.

PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
@aqrln
Copy link
Member Author

aqrln commented Apr 3, 2017

Landed in 52f273d, 8d66c79, bae8950, d40370b and e28334d.

@aqrln aqrln closed this Apr 3, 2017
@aqrln aqrln deleted the v0.5-backport-2 branch April 3, 2017 17:24
@aqrln
Copy link
Member Author

aqrln commented Apr 3, 2017

For the record: Travis and bitHound linter passing, bitHound dependency checker complains at stale webpack devDependency we cannot upgrade for v0.5.

@aqrln aqrln mentioned this pull request Apr 3, 2017
aqrln added a commit that referenced this pull request Apr 3, 2017
This is a partial backport of 4f83d92
and a backport of 60ae973 together with
that.

PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
aqrln added a commit that referenced this pull request Apr 3, 2017
This is a backport of eb49729.

PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
aqrln added a commit that referenced this pull request Apr 3, 2017
This is kind of backports PR #105 in a different way, and this way
might be even better than that the original commit used.

The reason of doing this for v0.5, besides fixing a bug, is the fact
that we cannot use metasync in v0.5 since it is not compatible with old
Node.js versions while old library versions that used to be are too
outdated and even get reported by bitHound as insecure.

Backport-of: #105
PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
aqrln added a commit that referenced this pull request Apr 3, 2017
Original commit: 16d8c85

PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
aqrln added a commit that referenced this pull request Apr 3, 2017
This is a backport of eafafd1,
6069f18 and
691795b.

PR-URL: #120
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants