-
Notifications
You must be signed in to change notification settings - Fork 114
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
Update CISCO-RTTMON-TC-MIB.my #109
Conversation
I don't have |
It is by error closing the text block which is supposed to be closed on line 438.
|
Totally missed that this is part of the existing issue #53 This one is just removing a double-quote so it should not have taken the team this long. As such I have poked the devs behind the internal defect I filed CSCwi58743. Also, the PR has to come from a Cisco Employee so I will need to close this; but I will link the issue when the PR comes across. |
What's the point of this repo then? It is so counter-intuitive to block all external contributions.. |
@Hipska, I understand the frustration. I would love to merge these typos type issues quickly and be done but if the source isn't updated we risk somebody else coming along with a different update that wipes a change like this out. External proposed PRs are fine, I just have to file an Cisco Defect to ensure the changes are picked up internally then replicated back out by the developer that owns the source files. Some further context: Historically, this repo replaced the public/anonymous GitHub was selected as the new location because it continues public access without the requirement of logging into a Cisco.com site. Plus it has nice new features like commit history and rendering of the support list html files via GitHub pages. |
Another nice feature is an easy way for external contributors to contribute fixes 😉 |
Typo fix
resolves #53