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

cml-publish mime and repo #519

Merged
merged 8 commits into from
May 18, 2021
Merged

Conversation

DavidGOrtega
Copy link
Contributor

@DavidGOrtega DavidGOrtega commented May 15, 2021

  • does not need repo for non native
  • Detects also non binary files.
  • Adds param mime-type to specify the mime in case the type is incorrect. I.E. application/json is still now detected as text/plain
  • Adds camelize cleaning the CLI interface
  • aliases gitlab-uploads
Usage: cml-publish.js <path to file>

      Options:
        --version                   Show version number                      [boolean]
        --md                        Output in markdown format [title || name](url).
                                                                             [boolean]
        --title, -t                 Markdown title [title](url) or ![](url title).
        --native, --gitlab-uploads  Uses driver's native capabilities to upload assets
                                    instead of CML's storage. Currently only available
                                    for GitLab CI.
        --rm-watermark              Avoid CML watermark.                     [boolean]
        --mime-type                 Specifies the mime-type. If not set guess it from
                                    the content.
        --file, -f                  Append the output to the given file. Create it if
                                    does not exist.
        --repo                      Specifies the repo to be used. If not specified is
                                    extracted from the CI ENV.
        --token                     Personal access token to be used. If not
                                    specified, extracted from ENV REPO_TOKEN,
                                    GITLAB_TOKEN, GITHUB_TOKEN, or BITBUCKET_TOKEN.
        --driver                    If not specify it infers it from the ENV.
                                                         [choices: \\"github\\", \\"gitlab\\"]
        -h                          Show help                                [boolean]"

closes #518
closes #480

@DavidGOrtega DavidGOrtega self-assigned this May 15, 2021
@DavidGOrtega DavidGOrtega added bug Something isn't working cml-publish Subcommand labels May 15, 2021
@DavidGOrtega DavidGOrtega removed bug Something isn't working cml-publish Subcommand labels May 15, 2021
@DavidGOrtega DavidGOrtega changed the base branch from master to release/v0.4.0 May 18, 2021 23:34
@DavidGOrtega DavidGOrtega merged commit 2eab018 into release/v0.4.0 May 18, 2021
@DavidGOrtega DavidGOrtega deleted the cml-publish-mime-no-repo branch May 18, 2021 23:35
DavidGOrtega added a commit that referenced this pull request May 19, 2021
* No await runner (#523)

* cml-publish mime and repo (#519)

* cml-publish mime and repo

* cleanup

* image/svg+xml

* buffer

* resolve

* text buffer outputs plain

* native bolean

* snapshot

* Fix pipe reading logic on Windows (#516)

The fs.readSync function throws EOF instead of returning 0 like in other systems.
See nodejs/node#35997 for more information.

* minor doc fixes (#534)

* cml-pr command (#510)

* test-cml-pr

* packages

* pre-release

* log isci

* log isci

* user_name

* log gitlab url

* log gitlab url

* short 8

* len paths

* no undef

* git

* files

* files

* files

* files

* files

* files

* files

* files

* files

* files

* files

* files

* files

* files

* files

* files

* files

* sha2

* sha2

* log filepath

* log filepath

* log filepath

* exists and  clean

* SHA vs SHA2

* pre-release

* pre-release 2

* pre-release 3

* pre-release 3

* pre-release 3

* pre-release 3

* pre-release 3

* pre-release 3

* git back

* git back

* git back

* release

* REPO_TOKEN

* html_url

* cml pr defaults

* cml pr defaults

* test strict again

* CI

* vars

* shorter params

* snapshots and olivaw

* test david

* BB fix

* snapshots

* No git sha dependencies (#517)

* check head_sha

* 0.4.0

Co-authored-by: Helio Machado <0x2b3bfa0+git@googlemail.com>
Co-authored-by: Casper da Costa-Luis <work@cdcl.ml>
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.

cml-publish does not support basic text files cml-publish fails in folders not init by git
3 participants