[YANGTOOLS-1193] Add the option to retain declaration reference in DeclaredStatement Created: 03/Dec/20 Updated: 23/Apr/21 Resolved: 23/Apr/21 |
|
| Status: | Resolved |
| Project: | yangtools |
| Component/s: | parser |
| Affects Version/s: | None |
| Fix Version/s: | 7.0.0 |
| Type: | New Feature | Priority: | Medium |
| Reporter: | Антон Трегубов | Assignee: | Антон Трегубов |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
This feature allows to create a navigation link to anchor in source file for "main" statements in YANGs Visualization UI.
This feature already implemented in https://github.com/anton-tregubov/yangtools/tree/6.0.x But I can't find any way to create Pull Request or Merge Request |
| Comments |
| Comment by Robert Varga [ 03/Dec/20 ] |
|
tregubov we are using Gerrit to modify the repos and unfortunately cannot accept GH PRs. https://wiki.opendaylight.org/display/ODL/Contributor+Guidelines has some links around that. |
| Comment by Антон Трегубов [ 03/Dec/20 ] |
|
Thx, i'll try |
| Comment by Антон Трегубов [ 07/Dec/20 ] |
|
https://git.opendaylight.org/gerrit/c/yangtools/+/94088 I think i'll done |
| Comment by Robert Varga [ 18/Dec/20 ] |
|
Looks good thanks. Unfortunately this ends up being API-incompatible and increases footprint for our common use case, hence we'll deliver it in 7.0.0 (~end of Feb 2021) |
| Comment by Антон Трегубов [ 19/Dec/20 ] |
|
Thx, This is a good news for me. Should i do something or just wait? https://git.opendaylight.org/gerrit/c/yangtools/+/94088 got label Merge Conflict. Sorry, i don't now all rules. |
| Comment by Robert Varga [ 23/Apr/21 ] |
|
No worries. Sorry for the late reply, your patch came in just as we were refactoring a lot of the things it was touching. |
