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

Update Contribute Document to Use Requirements.txt #470

Merged
merged 2 commits into from
Apr 30, 2016

Conversation

rcuza
Copy link
Contributor

@rcuza rcuza commented Apr 22, 2016

Made a minor simplification change to the CONTRIBUTE.md document. Instead of listing all the commands for installing modules, I put the modules into a requirements.txt file (via pip freeze) and then updated the docs on how to use the new file.

This change should help with code documentation rot.

One possible improvement is to use >= on the versioning, but would only want to do that if the CI tests always used the bleeding edge version.

Versions tested with python 2.7.
@philtay
Copy link
Contributor

philtay commented Apr 25, 2016

I think the dev dependencies listed in the docs/requirements.txt file should not be pinned to specific versions.

@rcuza
Copy link
Contributor Author

rcuza commented Apr 28, 2016

@philtay - Great point. I removed the version pinning.

@markpeek markpeek merged commit 344b2e1 into cloudtools:master Apr 30, 2016
@markpeek
Copy link
Member

Thanks!

webratz pushed a commit to webratz/troposphere that referenced this pull request May 4, 2016
* cloudtools/master: (80 commits)
  Add test for AWS::Logs::Destination (cloudtools#482)
  Fix baseclass for AWS::Logs::Destination (Fixes cloudtools#481)
  Update for changes since 1.5.0
  Fix Bucket AccessControl to allow Ref (cloudtools#475)
  Better AWSHelperFn support in template generator (cloudtools#473)
  Implement LifecycleRule Transitions property (cloudtools#472)
  Update to Apr 25, 2016 release (cloudtools#471)
  Fix Source object to take a list of SourceDetails (cloudtools#469)
  Update Contribute Document to Use Requirements.txt (cloudtools#470)
  Add WAF SizeConstraintSet and XssMatchSet
  Add backward compatibility for the deprecated ElasticsearchDomain
  Fix Elasticsarch Domain object naming
  Import JSON Templates (cloudtools#468)
  ReplicationConfigurationRules Destination is now an object (cloudtools#380)
  Remove unnecessary AWSHelperFn from props
  EC2 update FromPort, ToPort and Egress as optional
  Added CloudWatch Events support
  ApiGateway Resources (cloudtools#466)
  Provide Valid Lambda Function Memory Values for use in Parameters (cloudtools#449)
  move Groups to property, add policy template version (cloudtools#460)
  ...
amosshapira pushed a commit to amosshapira/troposphere that referenced this pull request Oct 24, 2016
Add requirements doc for python. Versions tested with python 2.7.
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

3 participants