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

chore: cleaning up docs and ci #7

Merged
merged 3 commits into from
Jan 25, 2024
Merged

Conversation

madflojo
Copy link
Member

Problem Statement

What is the current behavior? Why and how does it need to change?

Docs have an error, and the CI config is quite old and needs to be updated.

Description of Change

Please include a summary of the change and, if applicable, tag related issues, add code snippets or logs.

Updates the docs fixing #6 and the CI to use just actions.

Breaking Change

Is this a breaking change?

Caveats

Please list any caveats or special considerations for this change.

@neelamb6 neelamb6 self-assigned this Jan 24, 2024
Copy link
Contributor

@neelamb6 neelamb6 left a comment

Choose a reason for hiding this comment

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

changes looks good

@madflojo madflojo merged commit 0640d42 into americanexpress:main Jan 25, 2024
4 checks passed
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.

2 participants