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

added teraterm application #3158

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

Conversation

bryanpedini
Copy link

original source found by in #1560
commit inspired by @RocGod
checked validity of winget and choco installation targets
modified url and description

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Hotfix
  • Security patch
  • UI/UX improvement

Description

I understand the project does not currently accept new applications as of right now, tho I believe the current PR can remain open until such possibility again arise, where I can merge the then-up-to-date main branch and/or rebase my commit on that, and the following could then be closed.
This, because I strongly believe that for terminal people like myself, TeraTerm is just a plain better experience for serial consoles, as for example it automatically detects COM ports, whereas PuTTY does not. (only reason I use tt, ik)

Testing

The testing was done locally using Compile.ps1 -run, no error was reported, and the application installed fine. Further tests can be performed on the dev branch if deemed necessary.

Impact

No foreseable impact on the project or its performance/security/dependencies

Issue related to PR

Additional Information

N/A

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

original source found by in ChrisTitusTech#1560
commit inspired by @RocGod
checked validity of winget and choco installation targets
modified url and description
Copy link

@JamesMinoucha JamesMinoucha left a comment

Choose a reason for hiding this comment

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

Good idea!

@Cryostrixx
Copy link
Contributor

Good idea!

Hey, @JamesMinoucha! Thanks for approving the changes but just to give you a quick heads up: WinUtil is currently not accepting new applications at the moment, as such this PR will likely be closed by the maintainer. Thanks for your understanding! :)

@JamesMinoucha
Copy link

Good idea!

Hey, @JamesMinoucha! Thanks for approving the changes but just to give you a quick heads up: WinUtil is currently not accepting new applications at the moment, as such this PR will likely be closed by the maintainer. Thanks for your understanding! :)

Oh ok, thank's you

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.

3 participants