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

Readme-updated #34

Merged
merged 6 commits into from Oct 9, 2017

Conversation

Projects
None yet
4 participants
@pooyadav
Contributor

pooyadav commented Oct 6, 2017

No description provided.

@Toshbrown Toshbrown requested a review from yousefamar Oct 6, 2017

@yousefamar

Thanks for that, though actually those specs are also the wrong ones. The actual latest standard is at http://shop.bsigroup.com/upload/276605/PAS212-corr.pdf (I don't know why they make it so hard to find). To avoid broken links, maybe we could host it ourselves.

@mor1

This comment has been minimized.

Show comment
Hide comment
@mor1

mor1 Oct 6, 2017

mor1 commented Oct 6, 2017

@pooyadav

This comment has been minimized.

Show comment
Hide comment
@pooyadav

pooyadav Oct 6, 2017

Contributor

Okie, for the time-being I have added the link. But yes, adding specs in me-box/core-arbiter/documents is not a bad idea. I have just a concern about copyright issues. Can we just download and host on our local github repo?

Contributor

pooyadav commented Oct 6, 2017

Okie, for the time-being I have added the link. But yes, adding specs in me-box/core-arbiter/documents is not a bad idea. I have just a concern about copyright issues. Can we just download and host on our local github repo?

@mor1

This comment has been minimized.

Show comment
Hide comment
@mor1

mor1 Oct 6, 2017

mor1 commented Oct 6, 2017

@Toshbrown Toshbrown merged commit 5012c0c into me-box:master Oct 9, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
security/snyk No new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment