Skip to content

Conversation

@ivan-nginx
Copy link
Member

No description provided.

@ivan-nginx ivan-nginx added this to the v7.1.0 milestone Mar 29, 2019
@1v9
Copy link
Member

1v9 commented Mar 29, 2019

I have a question that I always want to ask! 😂

批注 2019-03-29 205136.jpg

@ivan-nginx ivan-nginx changed the title Contributors per line: 6 instead of 7. Contributors per line: 6 instead of 7 Mar 29, 2019
@ivan-nginx ivan-nginx merged commit 6738d90 into master Mar 29, 2019
@ivan-nginx ivan-nginx deleted the ivan-nginx-patch-1 branch March 29, 2019 14:01
@stevenjoezhang
Copy link
Contributor

Need to regenerate README? It's still 7 per line

@ivan-nginx
Copy link
Member Author

Yeah, it seems bot have some bugs. For example, if 2 contributors was added (2 pulls created) in second pull merge will conflict.

Moreover, maybe better will use not README.md file, but any another file? (can be defined in bot's config).

@stevenjoezhang
Copy link
Contributor

@ivan-nginx
Copy link
Member Author

ivan-nginx commented Mar 29, 2019

Maybe yes. I think must be named «MAINTAINERS.md» or «CONTRIBUTORS.md».

@stevenjoezhang and authors / contributors must be specified in commits / contributions order (top contributoes must be in top).

@stevenjoezhang
Copy link
Contributor

And maybe we can add more 3rd party applications to this part:

屏幕快照 2019-03-30 上午12 58 30

@ivan-nginx
Copy link
Member Author

See Telegram chat.

@ivan-nginx
Copy link
Member Author

I'll assigned to it in #758.

tongluyang pushed a commit to tongluyang/hexo-theme-next that referenced this pull request Nov 19, 2019
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.

4 participants