Skip to content
This repository was archived by the owner on Nov 9, 2022. It is now read-only.

Conversation

artasparks
Copy link

This updates the selector window with the changes to maktaba.

See: https://github.com/google/vim-maktaba/blob/master/autoload/maktaba/selector.vim

This updates the selector window with the changes to maktaba.

See: https://github.com/google/vim-maktaba/blob/master/autoload/maktaba/selector.vim
@artasparks artasparks requested a review from dbarnett May 27, 2020 20:36
Copy link
Contributor

@dbarnett dbarnett left a comment

Choose a reason for hiding this comment

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

If you're getting rid of the legacy branch but still want this plugin to exist, you could just make it a thin wrapper around maktaba#selector.

@@ -1,4 +1,4 @@
selector is a utility to help plugin authors launch a window to choose one of
# selector is a utility to help plugin authors launch a window to choose one of
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this "#" intentional? I think that turns it into a title, not commented out.

@@ -12,34 +12,18 @@
" See the License for the specific language governing permissions and
" limitations under the License.

""
" @section Introduction, intro
Copy link
Contributor

Choose a reason for hiding this comment

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

Should still have some kind of intro section, shouldn't it? The maktaba file won't have one because it would conflict with maktaba's own intro.

What edits do these source file changes write into the generated docs?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants