Skip to content

Fixed #155 - Add 'Comment Template' to comment logic block.#210

Closed
Minku-Koo wants to merge 23 commits intovisualpython:devopsfrom
Minku-Koo:devops
Closed

Fixed #155 - Add 'Comment Template' to comment logic block.#210
Minku-Koo wants to merge 23 commits intovisualpython:devopsfrom
Minku-Koo:devops

Conversation

@Minku-Koo
Copy link
Copy Markdown
Contributor

  • Add comment template on logic block
  • Templates are consisting of default, class and method

minjk-bl and others added 23 commits August 5, 2022 15:48
Add description for using Google Colab extension
Update installation manual and banners
Edit banners
Release v2.3.0 (Integrated version of all extension environments)
Copy link
Copy Markdown
Collaborator

@minjk-bl minjk-bl left a comment

Choose a reason for hiding this comment

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

Hello, thank you for the changes you've made, but your repository is not rebased with the current commits.

You should first fetch & rebase your local repo using

  • git fetch upstream
  • git rebase upstream/devops

Then push it to your forked-repository using

  • git push origin devops

In this pull-request, many past-codes are remaining yet. Merging Comment.js file is only needed for your request.

  • After you do fetch -> rebase -> push, then let us know to review this PR again. Thanks :)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This code should be included. This file should be rebased to current base.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This code should be included. This file should be rebased to current base.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This file should be rebased to current base

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This code should be included. This file should be rebased to current base.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This file should be rebased to current base.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This file should be rebased to current base.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This file should be rebased to current base.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This file should be rebased to current base.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This file should be rebased to current base.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It works well :)

@Minku-Koo
Copy link
Copy Markdown
Contributor Author

Thank you for your help.
I create new PR, so close this PR.

@Minku-Koo Minku-Koo closed this Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

need reply Need developer's reply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants