golang/go

x/tools/cmd/present2md: support full translation

Open

#37,917 opened on Mar 18, 2020

View on GitHub
 (2 comments) (1 reaction) (0 assignees)Go (19,008 forks)batch import
NeedsInvestigationToolshelp wanted

Repository metrics

Stars
 (133,883 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Not sure this should be considered part of #33955. Please close if so.

The new present2md tool converts the present format to markdown. However, it is still missing some pieces of the present format when converting, namely:

  • Captions
  • Images

There may be more types I've missed, but these were the ones I noticed so far.

These are being caught by the fallback PresentCmd interface here:

https://github.com/golang/tools/blob/11a475a590acad44f466dde7d3e988cf52bad01c/cmd/present2md/main.go#L236-L247

I think it should be possible to convert both of these to markdown, though the caption may be more complicated.

Contributor guide

Research direction
Study the present format specification and the existing present2md conversion logic. Identify how captions and images are represented in the present format and map them to markdown equivalents. Implement the missing cases in the switch statement or add new handlers for PresentCmd types.
Tech stack
go
Domain
cli
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Go
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
55