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

Humble sumission for function help #29

Open
neztach opened this issue Jun 24, 2024 · 12 comments
Open

Humble sumission for function help #29

neztach opened this issue Jun 24, 2024 · 12 comments

Comments

@neztach
Copy link

neztach commented Jun 24, 2024

I took curser.sh and leveraged it to add function help for O365Essentials in hopes that it may be of some use to you for documentation.

Please have a look and I sincerely hope it helps you and this project.
https://github.com/neztach/PS-Public/blob/master/evotec-O365Essentials.ps1

@PrzemyslawKlys
Copy link
Member

Hi,

I have no use for it if its not provided as PR and added directly to sources. Using my main psm1 and modifying it would mean i need to reapply whatever you created and spend lots of time on applying it to sources.

If you can learn PR on github I'm sure more docs would help.

@neztach
Copy link
Author

neztach commented Jun 24, 2024

I created a fork, and put my edits there. Will that work?

https://github.com/neztach/O365Essentials/tree/master

@PrzemyslawKlys
Copy link
Member

Yes, you then create PR from it:

image

However 2 things I would like to ask:

  1. Each file you remove and add last lines? Not sure if necessary?

image

  1. I believe the formatting looks better in this way? So basically one line white space and without additional tab would be better

image

@neztach
Copy link
Author

neztach commented Jun 24, 2024

submitted. I could go back and add the spacing if need be to the help as you indicated. I'm just trying to help out since I sincerely love the content you've released and I've learned SO much from you. I wanted to try to contribute and give back if possible.

Please let me know if this is a valuable contribution, and what alterations you'd like me to make other than writing help for the Functions I didn't already.

@PrzemyslawKlys
Copy link
Member

It is valuable contribution and you shouldn't even need to ask. I love when people help with my projects, even if it's basic things. I've loads of modules that lack help, I have loads of modules that require tests, I've loads of modules that requires fixes and improvements so if people want to help - that's great. I don't have time to do a lot of things, and sometimes I do just enough for things to work. I have plenty of modules that I've not even blogged about to spread the word that they exists. I started also adding readme , showing more usage scenarios and all the details - so if you want you can help with that as well.

Generally I'll take any help I can get with anything :)

I saw your latest commit and I believe you also need to fix this:

image

So basically remove the blue part. I'm sure it's valid comment but my modules usually use no space/tabs

image

I know it's a bit pain and not really important, as Get-Help will work either way, but it's just synthetic sugar for me :)

@neztach
Copy link
Author

neztach commented Jun 24, 2024

gotcha! OK I'll go through and remedy those, and then I guess submit another pull request?

@PrzemyslawKlys
Copy link
Member

PrzemyslawKlys commented Jun 24, 2024

no, when you commit changes they update the PR directly.

When you do a fork, and you have single branch, and you change something in it, and then do PR, as long as you stay within same branch

image

You will keep on adding files to this same branch, which will continue updating the PR. You can also mark the PR as draft until you're happy with results.

image

@neztach
Copy link
Author

neztach commented Jun 24, 2024

ok help updated as requested. Please let me know what else I can help with.

@PrzemyslawKlys
Copy link
Member

What about:

image

It seems to be in every file? Btw you don't need to commit per file, you can bundle it up under one commit ;)

@neztach
Copy link
Author

neztach commented Jun 26, 2024

I honestly have no idea. After I forked your repo, I went through the regular Github web interface and edited each file only to add the help, then saved. I didn't mess with any other parts of the code.

@PrzemyslawKlys
Copy link
Member

You should try VSCode - doing it via GitHub Web is painful. I'll take a look how to fix it.

@neztach
Copy link
Author

neztach commented Jun 27, 2024

pulled my fork into curser with git, finished adding help to functions I previously missed. Still don't see the double close curly brace. But you were right of course, single pull - make all changes - stange with comment - single push - easy.

At least I got through my previously missed function help.

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

No branches or pull requests

2 participants