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

[post][ayi] Ayi a common util cli #7

Merged
merged 4 commits into from
Jan 1, 2017
Merged

[post][ayi] Ayi a common util cli #7

merged 4 commits into from
Jan 1, 2017

Conversation

at15
Copy link
Member

@at15 at15 commented Dec 29, 2016

Related issue: N/A

Post info

  • Title: Ayi
  • Author: @at15
  • License: default
  • Estimate finish time: 12/29 PST 12/31 PST

Related posts

  • N/A

Related projects

  • Ayi A common util application

@at15 at15 self-assigned this Dec 29, 2016
@at15 at15 changed the title [post][ayi] Ayi ya yi ya [WIP][post][ayi] Ayi ya yi ya Dec 29, 2016

## 主要问题

- 人太少,基本只有 @at15 (我) 一个人
Copy link
Member

Choose a reason for hiding this comment

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

添加 GitHub 链接会好一些?

Copy link
Member Author

Choose a reason for hiding this comment

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

懒...这个应该ink的render支持....

Copy link
Member

Choose a reason for hiding this comment

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

是么,好吧,厉害了我的 ink

Copy link
Member Author

Choose a reason for hiding this comment

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

no, I mean ... ink should have supported it...

Copy link
Member

Choose a reason for hiding this comment

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

。。。。。。。。

Copy link
Member

Choose a reason for hiding this comment

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

可以mos开个坑=。= ink github支持插件=,=

Copy link
Member

Choose a reason for hiding this comment

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

@JasonQSY 资磁~ 开个吧~

Copy link
Member

Choose a reason for hiding this comment

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

go to dyweb/mos#12

- 文件传输
- go 的版本从 1.5 跳到了 1.7。原生支持 vendor 并且有了很多更好的依赖管理工具,比如 [glide](https://github.com/Masterminds/glide)

第一个问题的话,基本无解,目前东岳经常写 Golang 的人好像只有我和策策。策策有空就要去陪妹子,自然不可能陪我来填坑。
Copy link
Member

Choose a reason for hiding this comment

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

俗话说得好,万事靠自己

Copy link
Member Author

Choose a reason for hiding this comment

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

所以我们也不会去帮助 processing.R 的 /w


## 总结

- 等有钱了,给大家都配 MBP
Copy link
Member

Choose a reason for hiding this comment

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

这个好


- 等有钱了,给大家都配 MBP


Copy link
Member

Choose a reason for hiding this comment

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

多了一个换行

Copy link
Member

@JasonQSY JasonQSY left a comment

Choose a reason for hiding this comment

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

🐑


## 主要问题

- 人太少,基本只有 @at15 (我) 一个人
Copy link
Member

Choose a reason for hiding this comment

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

可以mos开个坑=。= ink github支持插件=,=

- go 的版本从 1.5 跳到了 1.7。原生支持 vendor 并且有了很多更好的依赖管理工具,比如 [glide](https://github.com/Masterminds/glide)

第一个问题的话,基本无解,目前东岳经常写 Golang 的人好像只有我和策策。策策有空就要去陪妹子,自然不可能陪我来填坑。
(要有妹子的话我还会去填坑么?)
Copy link
Member

Choose a reason for hiding this comment

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

单身还是第一生产力 👍

- 使用 `git log -reverse` 可以反过来看log, 可以用来找第一个提交。
- shell 在 windows 下基本不会有问题,因为为了使用 git,东岳所有的 windows 用户都安装了
msysgit (现在叫 git for windows),它自带了 bash 和一些基本的工具。
- 周报的功能作为 MOS 的一个项目交给了 @codewarm96, 进度见[这个issue](https://github.com/dyweb/mos/issues/1)
Copy link
Member

Choose a reason for hiding this comment

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

拼错了=。= @codeworm96

Copy link
Member

Choose a reason for hiding this comment

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

@JasonQSY
Copy link
Member

感觉pull approve的体验不错。。(感觉我就是来玩pullapprove的

@gaocegege
Copy link
Member

WIP 说吼是不管用的。。

@JasonQSY
Copy link
Member

wip去掉以后重判嘛。。

@gaocegege
Copy link
Member

好像是的

@at15 at15 added the delayed label Dec 30, 2016
@arrowrowe
Copy link
Member

arrowrowe commented Dec 30, 2016

🐑

Approved with PullApprove

- 使用 `git log -reverse` 可以反过来看log, 可以用来找第一个提交。
- shell 在 windows 下基本不会有问题,因为为了使用 git,东岳所有的 windows 用户都安装了
msysgit (现在叫 git for windows),它自带了 bash 和一些基本的工具。
- 周报的功能作为 MOS 的一个项目交给了 @codeworm96, 进度见[这个issue](https://github.com/dyweb/mos/issues/1)
Copy link
Member

Choose a reason for hiding this comment

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

@ComMouse
Copy link
Member

ComMouse commented Dec 31, 2016

LGTM

Approved with PullApprove

- Add encountered problem when using open source libraries
- Add common util
- Fix another Codeworm typo (that is not fixed by last commmit)  @codeworm96 /w\
- Add roadmap (sort of)
@at15 at15 changed the title [WIP][post][ayi] Ayi ya yi ya [post][ayi] Ayi ya yi ya Dec 31, 2016
@at15 at15 changed the title [post][ayi] Ayi ya yi ya [post][ayi] Ayi a common util cli Dec 31, 2016
@at15
Copy link
Member Author

at15 commented Dec 31, 2016

I tried to add toc using https://github.com/jonschlinkert/markdown-toc, but ink is using https://github.com/russross/blackfriday which does not add id for header, so I would just give up for now.....

It is possible to switch to https://github.com/shurcooL/github_flavored_markdown in ink and external toc generation would work

@at15 at15 removed the delayed label Dec 31, 2016
@JasonQSY
Copy link
Member

JasonQSY commented Jan 1, 2017

LGTM

@JasonQSY
Copy link
Member

JasonQSY commented Jan 1, 2017

好像还是没用到 blog group =。= @at15

@at15
Copy link
Member Author

at15 commented Jan 1, 2017

@JasonQSY seems to be a bug with pullapprove, collaborators are not synchronized.... I still see the old team members as potential reviewers in pullapprove's page ...

maybe create a issue here https://github.com/pullapprove/support ?

@gaocegege
Copy link
Member

gaocegege commented Jan 1, 2017

LGTM

Approved with PullApprove

@arrowrowe
Copy link
Member

arrowrowe commented Jan 1, 2017

🐑

Approved with PullApprove

@JasonQSY
Copy link
Member

JasonQSY commented Jan 1, 2017

Know little about the configuration of pullapprove.. Maybe some one configuring it report the problem? @at15

@at15 at15 merged commit 409aeab into master Jan 1, 2017
@gaocegege gaocegege deleted the post/ayi branch January 2, 2017 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants