Skip to content

Edit doc to comply with the go one #7

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

Merged
merged 3 commits into from
Mar 2, 2023
Merged

Edit doc to comply with the go one #7

merged 3 commits into from
Mar 2, 2023

Conversation

lucasscw
Copy link
Contributor

@lucasscw lucasscw commented Mar 2, 2023

Summary

What's changed?

  • Add several part from the go library

Why do we need this?

How have you tested it?

Checklist

  • I have reviewed this myself
  • There is a unit test covering every change in this PR
  • I have updated the relevant documentation

Details

@lucasscw lucasscw requested a review from cyclimse March 2, 2023 15:13
README.md Outdated

## 🎓 Contributing

We welcome all contributions to our open-source projects, please see our [contributing guidelines](.github/CONTRIBUTING.md).
We welcome all contributions to our open-source projects, please see our contributing guidelines <link>.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
We welcome all contributions to our open-source projects, please see our contributing guidelines <link>.
We welcome all contributions to our open-source projects, please see our contributing guidelines [contributing guidelines](.github/CONTRIBUTING.md).

README.md Outdated
Comment on lines 96 to 99
## 🧑‍💻 Development

This repository is at its early stage and is still in active development.
If you are looking for a way to contribute please read [CONTRIBUTING.md](./docs/CONTRIBUTING.md).
Copy link
Collaborator

Choose a reason for hiding this comment

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

TBH, I think this section is too similar to the Contributing section

README.md Outdated
Comment on lines 64 to 117
We love feedback. Feel free to:

- Open a [Github issue](https://github.com/scaleway/serverless-functions-python/issues/new)
- Send us a message on the [Scaleway Slack community](https://slack.scaleway.com/), in the [#serverless-functions](https://scaleway-community.slack.com/app_redirect?channel=serverless-functions) channel.
We love feedback.
Don't hesitate to open a [Github issue](https://github.com/scaleway/serverless-functions/go/issues/new) or
feel free to reach us on [Scaleway Slack community](https://slack.scaleway.com/),
we are waiting for you on [#serverless-functions](https://scaleway-community.slack.com/app_redirect?channel=serverless-functions).
Copy link
Collaborator

Choose a reason for hiding this comment

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

This differs based on the feedback from Simon: #1 (comment)

@cyclimse cyclimse requested a review from thomas-tacquet March 2, 2023 17:23
@cyclimse cyclimse merged commit dbd1742 into main Mar 2, 2023
@cyclimse cyclimse deleted the lucasscw-patch-1 branch March 2, 2023 17:24
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