Skip to content

adds CHANGELOG entry for PR 2322#2325

Merged
bobbrow merged 2 commits into
microsoft:masterfrom
john-patterson:jopatte/2322-changlog-entry
Jul 27, 2018
Merged

adds CHANGELOG entry for PR 2322#2325
bobbrow merged 2 commits into
microsoft:masterfrom
john-patterson:jopatte/2322-changlog-entry

Conversation

@john-patterson

Copy link
Copy Markdown
Contributor

Realized I had left out the CHANGELOG update. For PR 2322.

@john-patterson john-patterson force-pushed the jopatte/2322-changlog-entry branch from 1538b7e to fd50c99 Compare July 25, 2018 04:00
Comment thread Extension/CHANGELOG.md Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We usually just try to do one line per github issue. I would just keep this first line and remove the rest.

Realized I had left out the CHANGELOG update. For [PR
2322](microsoft#2322).
@john-patterson john-patterson force-pushed the jopatte/2322-changlog-entry branch from fd50c99 to 3a7da84 Compare July 25, 2018 04:04
@sean-mcmanus

sean-mcmanus commented Jul 25, 2018

Copy link
Copy Markdown
Contributor

What was the 2nd line? I don't see it.

@john-patterson

Copy link
Copy Markdown
Contributor Author

@sean-mcmanus I had overwritten it with push -f, I had a line for the following:

* Fixed unit test task for VS Code in launch.json.
* Fixed unit test gulp target to be runnable form Travis CI.
* Fixed bug allowing for variables with "env", "config", and "workspaceFolder"

@bobbrow bobbrow merged commit 84ff661 into microsoft:master Jul 27, 2018
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants