Skip to content

Update README.md, fix the license name in setup.py#105

Merged
skupriienko merged 2 commits into
mailjet:masterfrom
oss-repack:docs/update_readme
Oct 11, 2024
Merged

Update README.md, fix the license name in setup.py#105
skupriienko merged 2 commits into
mailjet:masterfrom
oss-repack:docs/update_readme

Conversation

@skupriienko
Copy link
Copy Markdown
Collaborator

@skupriienko skupriienko commented Sep 23, 2024

Actions:

  • Fix the api_credential URL
  • Use api_key and api_secret lowercased consistently
  • Use print() for Python 3 support with a suggestion Python 2 will be dropped in the next PR
  • Update an example in Make your first call to fix README is outdated #89
  • Fix the API Reference URL
  • Use id_ instead of builtin id
  • Fix the license name in setup.py

Copy link
Copy Markdown

@DanyilNefodov DanyilNefodov left a comment

Choose a reason for hiding this comment

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

LGTM overall

Comment thread setup.py Outdated
@skupriienko skupriienko merged commit f3a3620 into mailjet:master Oct 11, 2024
@skupriienko skupriienko deleted the docs/update_readme branch October 11, 2024 05:40
skupriienko added a commit to oss-repack/mailjet-apiv3-python that referenced this pull request May 5, 2025
Update README.md, fix the license name in setup.py
@skupriienko skupriienko self-assigned this May 5, 2025
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.

README is outdated

2 participants