Skip to content
This repository has been archived by the owner on Sep 13, 2022. It is now read-only.

Add Elixir migration example #71

Closed
wants to merge 2 commits into from
Closed

Add Elixir migration example #71

wants to merge 2 commits into from

Conversation

yahonda
Copy link
Member

@yahonda yahonda commented Jan 17, 2022

First-time contributors' checklist

What is changed, added or deleted? (Required)

Add Elixir migration example by following this getting-started instructions to use TiDB
https://hexdocs.pm/ecto/getting-started.html

Which TiDB version(s) do your changes apply to? (Required)

What is the related PR or file link(s)?

  • This PR is translated from:
  • Other reference link(s):

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases

- Following this getting-started instructions to use TiDB
https://hexdocs.pm/ecto/getting-started.html
@ti-chi-bot
Copy link
Member

[REVIEW NOTIFICATION]

This pull request has not been approved.

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

@ti-chi-bot ti-chi-bot added first-time-contributor Indicates that the PR was contributed by an external member and is a first-time contributor. missing-translation-status size/L labels Jan 17, 2022
@yahonda
Copy link
Member Author

yahonda commented Jan 17, 2022

Let me take a look at CI lint failures.

en/for-ecto.md:151 MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```sql"]
en/for-ecto.md:222 MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]
@yahonda yahonda closed this Sep 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
first-time-contributor Indicates that the PR was contributed by an external member and is a first-time contributor. missing-translation-status size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants