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

initial changes for 1.80 release notes/Readme.txt file #40

Merged
merged 8 commits into from
Oct 5, 2022

Conversation

w7sst
Copy link
Owner

@w7sst w7sst commented Oct 1, 2022

Hi @scotthibbs, @ct7aup,
I'm trying to create a code review so we can work together to finish these release notes. If this step correctly creates a code review, I think you are able to make additional changes within the code review itself without loading the file local. If you take the file local, you can push your change back to the same branch. Once we are done with our changes, we can merge this branch to main by finalizing this merge request. Initially this is getting created as a draft pull request.

@w7sst w7sst linked an issue Oct 1, 2022 that may be closed by this pull request
Copy link
Owner Author

@w7sst w7sst left a comment

Choose a reason for hiding this comment

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

@scotthibbs, @ct7aup
To see the file, you have to click on the Files changed link above. Then you can edit the file by clicking on the ... button in the upper right of the readme.txt file window. Please make any additional edits and save the file back on this branch.

Let me know if this technique does not work and we can try this a different way.

Thanks,
Mike

@scotthibbs
Copy link
Collaborator

scotthibbs commented Oct 3, 2022 via email

A few more minor changes. Add some missing log error checks.
Readme.txt Show resolved Hide resolved
Readme.txt Show resolved Hide resolved
Readme.txt Outdated Show resolved Hide resolved
Copy link
Collaborator

@scotthibbs scotthibbs left a comment

Choose a reason for hiding this comment

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

I added the version number to the top of the readme so we know we have the right readme for each version. I also added the date to the copyright. I changed the "I" in a paragraph that Alex Shovkoplyas, VE3NEA wrote so I changed it to reflect it was him.

So I think we got it. - Scott

@w7sst w7sst marked this pull request as ready for review October 5, 2022 18:08
@w7sst w7sst merged commit aefbe55 into main Oct 5, 2022
@w7sst w7sst deleted the 39-update-readmetxt-for-180-release branch October 5, 2022 18:11
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.

Update Readme.txt for 1.80 release
3 participants