Skip to content

Port changes for release v1.0.19 and Changelog #111

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

Merged
merged 10 commits into from
Aug 13, 2020
Merged

Conversation

sfsholden
Copy link
Contributor

What does this PR do?

  1. Updated changelog for v1.0.19
  2. Port commits from develop to main

@sfsholden sfsholden requested a review from a team as a code owner August 13, 2020 16:45
Apply suggestions from code review

Co-authored-by: Bryan Powell <b.powell@salesforce.com>
@lcampos
Copy link
Contributor

lcampos commented Aug 13, 2020

@sfsholden @brpowell I think this PR looks good. Just to confirm, we are not porting PR #112 as part of this since it just got merged earlier today, right ?

@brpowell
Copy link
Contributor

brpowell commented Aug 13, 2020

@sfsholden @brpowell I think this PR looks good. Just to confirm, we are not porting PR #112 as part of this since it just got merged earlier today, right ?

@sfsholden Actually, can that commit be included as well?

@sfsholden
Copy link
Contributor Author

sfsholden commented Aug 13, 2020

@brpowell Yes, I'm going to hold off on porting it until the QA is complete - just in case there are additional changes needed.

@sfsholden
Copy link
Contributor Author

@brpowell Just got the go ahead from Jag. I'll port that change now

@sfsholden sfsholden merged commit fe9c9f7 into main Aug 13, 2020
@sfsholden sfsholden deleted the sh/port-v1.0.19 branch August 13, 2020 20:17
@autointegrator
Copy link
Contributor

⚠️ The changes in the cherry-pick are already present in the develop branch. No port necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants