Skip to content

Feature/30 copy readme#34

Merged
TimPurdum merged 3 commits into
developfrom
feature/30_copy_readme
Sep 26, 2022
Merged

Feature/30 copy readme#34
TimPurdum merged 3 commits into
developfrom
feature/30_copy_readme

Conversation

@TimPurdum
Copy link
Copy Markdown
Collaborator

Closes #30
Prevents having to maintain two copies of ReadMe
On build, copies ReadMe into location for the Nuget Packaging.

@TimPurdum TimPurdum requested a review from seahro September 24, 2022 16:18
@TimPurdum TimPurdum self-assigned this Sep 24, 2022
Copy link
Copy Markdown
Collaborator

@seahro seahro left a comment

Choose a reason for hiding this comment

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

added NPM install reference and readme reference, remainder were reformatting

<Exec ContinueOnError="true" Command="robocopy ../../ ./ ReadMe.md /it"/>
</Target>

<Target Name="NPM Install" AfterTargets="Copy Readme">
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks like a reformatted file with a single change (NPM Intall?)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It's actually the Copy Readme command added, and the NPM Install, which was already there, got shoved downward.

Comment thread .gitignore
/src/.idea/**/*
/src/dymaptic.GeoBlazor.Core/wwwroot/js/* No newline at end of file
/src/dymaptic.GeoBlazor.Core/wwwroot/js/*
/src/dymaptic.GeoBlazor.Core/ReadMe.md No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

added readme...looks good

@TimPurdum TimPurdum merged commit ee48e12 into develop Sep 26, 2022
@TimPurdum TimPurdum deleted the feature/30_copy_readme branch September 26, 2022 13:27
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.

Make ReadMe.md copy into nuget package on build.

2 participants