Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Use Microsoft.PowerShell.MarkdownRender package from nuget.org#14090

Merged
rjmholt merged 3 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
adityapatwardhan:RemoveMarkRenderadityapatwardhan/PowerShell:RemoveMarkRenderCopy head branch name to clipboard
Dec 9, 2020
Merged

Use Microsoft.PowerShell.MarkdownRender package from nuget.org#14090
rjmholt merged 3 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
adityapatwardhan:RemoveMarkRenderadityapatwardhan/PowerShell:RemoveMarkRenderCopy head branch name to clipboard

Conversation

@adityapatwardhan

@adityapatwardhan adityapatwardhan commented Nov 13, 2020

Copy link
Copy Markdown
Member

PR Summary

The code for Microsoft.PowerShell.MarkdownRender has been moved to a separate repository https://github.com/powershell/markdownrender. This PR removes the code from this repository and uses the package from nuget.org instead

PR Context

PR Checklist

@ghost ghost assigned rjmholt Nov 13, 2020
@xtqqczze

xtqqczze commented Nov 14, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor

I see the initial commit PowerShell/MarkdownRender@345e2f3 was on 2020-10-02. We should ensure changes made in the last few months are reflected in powershell/markdownrender:

TravisEz13
TravisEz13 requested changes Nov 19, 2020
View reviewed changes
Comment thread tools/packaging/packaging.psm1 Outdated

@TravisEz13 TravisEz13 Nov 19, 2020

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.

This file should be signed in your build, so can you remove it from our signing?

@ghost ghost added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Nov 19, 2020
@ghost ghost removed the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Nov 20, 2020
@adityapatwardhan

adityapatwardhan commented Nov 20, 2020

Copy link
Copy Markdown
Member Author

@TravisEz13 - please re-review

@adityapatwardhan

adityapatwardhan commented Nov 20, 2020

Copy link
Copy Markdown
Member Author

@rjmholt This is ready for review now.

@adityapatwardhan adityapatwardhan mentioned this pull request Nov 20, 2020
Closed
@adityapatwardhan

adityapatwardhan commented Nov 20, 2020

Copy link
Copy Markdown
Member Author

@xtqqczze - Filed issue in MarkdownRender repo to make sure the improvements mentioned above are taken there: PowerShell/MarkdownRender#11

@adityapatwardhan

adityapatwardhan commented Nov 24, 2020

Copy link
Copy Markdown
Member Author

@rjmholt This is ready to merge.

@ghost ghost added the Review - Needed The PR is being reviewed label Dec 2, 2020
@ghost

ghost commented Dec 2, 2020

Copy link
Copy Markdown

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

@xtqqczze

xtqqczze commented Dec 2, 2020

Copy link
Copy Markdown
Contributor

@iSazonov We should avoid merging any more changes to Microsoft.PowerShell.MarkdownRender, like those in #14078.

@iSazonov

iSazonov commented Dec 2, 2020

Copy link
Copy Markdown
Collaborator

@iSazonov We should avoid merging any more changes to Microsoft.PowerShell.MarkdownRender, like those in #14078.

These files will simply be deleted - this does not affect anything.

@ghost ghost removed the Review - Needed The PR is being reviewed label Dec 2, 2020
@ghost ghost added the Review - Needed The PR is being reviewed label Dec 9, 2020
@ghost

ghost commented Dec 9, 2020

Copy link
Copy Markdown

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

@rjmholt

rjmholt commented Dec 9, 2020

Copy link
Copy Markdown
Collaborator

@PoshChan please remind me in 1 hour

@ghost ghost removed the Review - Needed The PR is being reviewed label Dec 9, 2020
@rjmholt rjmholt added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label Dec 9, 2020
@rjmholt rjmholt merged commit 600b658 into PowerShell:master Dec 9, 2020
@rjmholt rjmholt deleted the RemoveMarkRender branch December 9, 2020 20:29
@PoshChan

PoshChan commented Dec 9, 2020

Copy link
Copy Markdown
Collaborator

@rjmholt, this is the reminder you requested 1 hour ago

@iSazonov iSazonov added this to the 7.2.0-preview.2 milestone Dec 10, 2020
@rjmholt rjmholt mentioned this pull request Dec 10, 2020
Merged
14 tasks
@ghost

ghost commented Dec 15, 2020

Copy link
Copy Markdown

🎉v7.2.0-preview.2 has been released which incorporates this pull request.:tada:

Handy links:

@xtqqczze xtqqczze mentioned this pull request Jun 29, 2021
Open
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TravisEz13 TravisEz13 TravisEz13 approved these changes

@daxian-dbw daxian-dbw Awaiting requested review from daxian-dbw

@anmenaga anmenaga Awaiting requested review from anmenaga

Assignees

@rjmholt rjmholt

Labels

CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log

Projects

None yet

Milestone

7.2.0-preview.2

Development

Successfully merging this pull request may close these issues.

6 participants

@adityapatwardhan @xtqqczze @iSazonov @rjmholt @PoshChan @TravisEz13
Morty Proxy This is a proxified and sanitized view of the page, visit original site.