Skip to content
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

Separate modules #12

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Separate modules #12

wants to merge 11 commits into from

Conversation

OgJAkFy8
Copy link
Contributor

Separated out the modules from the subnet tools, because they were more about testing something. Where the rest were more about getting information.

Suggest making the following changes to the structure since it is getting more complex.
GetIPv4Address\Modules - This will hold the modules
GetIPv4Address\Testing - For Pester
GetIPv4Address\GetIPv4Address.psd1 - the manifest file

Separated out the modules from the subnet tools, because they were more about testing something.  Where the rest were more about getting information.
Updated the manifest file with the new scripts, but didn't update the version or the GUID.
Some formatting changes.  Moved the primary function to the top of the page.
moved the files around and figured out the issue with the nested modules syntax.
Recreated the update file, so that it builds the manifest file correctly.  It must have had a typo before.
Using the new manifest update script
More formatting cleanup to line up the list with the location in the document and put the "converts" together.  Added a write up for the main module.
Added the writeup from the actual website.
Fixed the MTU output.
Changed remvove-varable parameter from Global to script to get rid of error message.
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.

1 participant