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

Minor spelling change #485

Merged
merged 2 commits into from Oct 6, 2020
Merged

Conversation

mflaxman
Copy link
Contributor

@mflaxman mflaxman commented Oct 5, 2020

I'm testing doing this all via github's web tool, it's unclear how this will affect git commit history (I believe everything will be automatically squashed down nicely).

Testing doing this via web tool, unclear how this will affect git flow
@k9ert k9ert merged commit 98bbcf2 into cryptoadvance:master Oct 6, 2020
@k9ert
Copy link
Collaborator

k9ert commented Oct 6, 2020

The annoying thing here is that PR-builds are only taken from whitelisted people (scam-detection-code trom travis-ci is detecting crypto-mining-code in the project but base the block on the person who did the PR) and you haven't so far. Will probably write yet another mail to travis in order to get the people whitelisted which are making more frequent changes.

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.

None yet

2 participants