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

feat: make api independent from each other #28

Merged
merged 1 commit into from
Apr 7, 2024
Merged

feat: make api independent from each other #28

merged 1 commit into from
Apr 7, 2024

Conversation

kiraware
Copy link
Owner

@kiraware kiraware commented Apr 7, 2024

This will make all API could be called using async with style rather than dependency with the BMKG class. This also make API more flexible.

this will make all api could be called using `async with` style rather
than dependency with the BMKG class. this also make api more flexible.
@kiraware kiraware added enhancement New feature or request python Pull requests that update Python code labels Apr 7, 2024
@kiraware kiraware requested a review from cexra April 7, 2024 04:50
@kiraware kiraware self-assigned this Apr 7, 2024
@kiraware kiraware changed the title make api independent from each other feat: make api independent from each other Apr 7, 2024
Copy link

codecov bot commented Apr 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (3baf0ab) to head (27bc9f9).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #28   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           48        48           
  Lines          661       662    +1     
=========================================
+ Hits           661       662    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@cexra cexra left a comment

Choose a reason for hiding this comment

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

lgtm

@cexra cexra merged commit 3dd019c into main Apr 7, 2024
10 checks passed
@cexra cexra deleted the update-api branch April 7, 2024 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants