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

changing back #1089

Merged
merged 1 commit into from
Jun 26, 2018
Merged

changing back #1089

merged 1 commit into from
Jun 26, 2018

Conversation

kenman345
Copy link
Member

@kenman345 kenman345 commented Jun 26, 2018

didnt help the dead url issue on travis. putting back to main level site

@EchterAgo
Copy link

The cointiger link seems to work fine for me. The reason I asked about it is because I take care to remove URLs that go to a specific language. Like https://some.site/en/index.html, there I would put https://some.site/ to make language specific redirects take you to the right site.

@kenman345
Copy link
Member Author

no I agree, I wanted to get rid of the error on the link validations and thought that would fix it but it did not, so changing it back

@EchterAgo
Copy link

Oh, I wasn't aware there was an error. I guess I should get a Linux build environment set up sometime so that I can run full rake.

@kenman345
Copy link
Member Author

I still havent done that either. I run in Windows and can run everything but the HTML proofer stuff. I use travis output for those.

@kenman345
Copy link
Member Author

Though I am getting a rPi 3B+ setup tonight to run docker so I can play around with some stuff. I wanna see about making use of HTTP2 Server Push stuff but havent had any other way to test changes for that.

@EchterAgo
Copy link

How can I see those without my tests failing?

@kenman345 kenman345 merged commit a993f23 into master Jun 26, 2018
@kenman345 kenman345 deleted the kenman345-patch-1 branch June 26, 2018 14:10
@kenman345
Copy link
Member Author

the second job on travis-ci is the ones that are allowed to fail. Thats the one that validates the external links

@kenman345
Copy link
Member Author

https://travis-ci.org/acceptbitcoincash/acceptbitcoincash/jobs/396904430

We only have that second job on PRs though. it takes time and why bother with every push when its allowed to fail anyways.

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