Skip to content

checking Lambda function into version control#133

Merged
SupahNickie merged 6 commits intoIABTechLab:masterfrom
SupahNickie:33-consent-passing
Jun 26, 2018
Merged

checking Lambda function into version control#133
SupahNickie merged 6 commits intoIABTechLab:masterfrom
SupahNickie:33-consent-passing

Conversation

@SupahNickie
Copy link
Copy Markdown
Contributor

Addresses #33

There is a versioning system on AWS, but it would good for documentation's sake to keep it here as well.

@SupahNickie SupahNickie requested a review from samtingleff June 22, 2018 23:22
Comment thread src/lambdaFunction.js Outdated
headers: {
version: [{
key: 'Version',
value: '13'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this Version header useful to keep around?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm not entirely certain; would it just go to $LATEST if it's not there?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm removing it; seems like it wouldn't really net us anything and would be a nightmare to keep updating in two places if we did make changes.

Copy link
Copy Markdown
Contributor

@samtingleff samtingleff left a comment

Choose a reason for hiding this comment

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

💯

@SupahNickie SupahNickie merged commit 86cc621 into IABTechLab:master Jun 26, 2018
@SupahNickie SupahNickie deleted the 33-consent-passing branch June 26, 2018 23:43
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.

2 participants