Skip to content

Update changelog with entries for PRs removing entire subpackages#180

Closed
aaronayres35 wants to merge 1 commit into
masterfrom
update-changelog-deleted-subpackages
Closed

Update changelog with entries for PRs removing entire subpackages#180
aaronayres35 wants to merge 1 commit into
masterfrom
update-changelog-deleted-subpackages

Conversation

@aaronayres35
Copy link
Copy Markdown
Contributor

In PRs #172, #173, and #175 the appscripting, template, and permissions subpackages were removed. This PR simply updates the changelog to record these changes for the 5.0.0 release.

@aaronayres35
Copy link
Copy Markdown
Contributor Author

Ah, I just now am seeing the slack discussion. We may not want to go through with this PR if we are instead going with an upcoming_changes folder approach

Copy link
Copy Markdown
Contributor

@rahulporuri rahulporuri left a comment

Choose a reason for hiding this comment

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

LGTM. I wasn't sure if we were linking to issues or PRs but it looks like the changelog for 4.5.0 links to PRs so this is good.

@rahulporuri
Copy link
Copy Markdown
Contributor

@aaronayres35 can you hold off on merging until we make a decision on #181 ?

@aaronayres35
Copy link
Copy Markdown
Contributor Author

Closing as #181 was merged

@rahulporuri rahulporuri deleted the update-changelog-deleted-subpackages branch November 5, 2020 13:51
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