Skip to content

Readme Update#6816

Merged
JuanPabloDiaz merged 7 commits intotech-conferences:mainfrom
JuanPabloDiaz:readme
May 31, 2024
Merged

Readme Update#6816
JuanPabloDiaz merged 7 commits intotech-conferences:mainfrom
JuanPabloDiaz:readme

Conversation

@JuanPabloDiaz
Copy link
Collaborator

This is how the README file looks after making the changes. What do you think?

The new contributors section seems to have too many contributors, we can limit them to 20 or so.

image


 Thanks for creating a new Pull Request for this repository.

To get the conferce as fast as possible into confs.tech please consider the following things.

Checklist for your change

  • does not delete another conference
  • has passed the tests via npm run test
  • has the correct order via npm run reorder-confs
  • is a developer conference: more than 3-4 people from different companies
  • is not a webinar, hackathon, marketing, zoom meeting with one person
  • URLs are reachable, as short as possible, do not contain tracking parameters and the website is dedicated for this event
  • topic is correct
  • conference name is as short as possible and does not include location and date

@JuanPabloDiaz
Copy link
Collaborator Author

Looking better now

image

Copy link
Member

@nimzco nimzco left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thank you for doing that 🙏🏼

Co-authored-by: Nima Izadi <[email protected]>
@cgrail
Copy link
Member

cgrail commented May 31, 2024

lgtm 👍 - thanks a lot

Copy link
Collaborator Author

@JuanPabloDiaz JuanPabloDiaz left a comment

Choose a reason for hiding this comment

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

LGTM

@JuanPabloDiaz JuanPabloDiaz merged commit 94a82cf into tech-conferences:main May 31, 2024
@JuanPabloDiaz JuanPabloDiaz deleted the readme branch July 4, 2024 19:22
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.

3 participants