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

Updating Folder and adding ReadME #66

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

amccann-Tek
Copy link
Contributor

@amccann-Tek amccann-Tek commented May 30, 2024

Pull Request Checklist

Please review the Contribution Guidelines before submitting.

  • Pulling against the master branch (left side).
  • You have previously submitted a Contributor License Agreement or have contacted a maintainer to request one.

Type (Select only one)

  • Bug fix
  • Working example
  • In-progress example (need Tektronix help)
  • In-progress example (does not need Tektronix help)

Description

Need a working link before we can publish application note, so wanted to get the folder set up for those purposes. The script itself will be uploaded later. READMEs will be updated later as well.

@amccann-Tek
Copy link
Contributor Author

@adametry @dwyban is there any opportunity for review today?

@dwyban
Copy link
Member

dwyban commented Jun 3, 2024

Hi @amccann-Tek . The new readme file says to see the full write up on tek.com at the link below, but there is no link.

WBG-DPT Written Guide on Tek.com does not have a link on it.

@amccann-Tek
Copy link
Contributor Author

@dwyban Correct. in a bit of a chicken and egg situation. They want to link here, we want to link there. Since files in here go through less review and publishing hoops I think we should stub ours out to get a link for them to include in their WBG app note. And then we can update these once published.

@dwyban
Copy link
Member

dwyban commented Jun 3, 2024

@amccann-Tek , got it. You're basically preparing the directory so we can have proper links.

Approving pull request.

@dwyban dwyban merged commit 69740d6 into tektronix:master Jun 3, 2024
3 checks passed
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.

2 participants