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

close #110, remove outdated config.env #115

Merged
merged 1 commit into from
Mar 6, 2024
Merged

Conversation

532910
Copy link
Contributor

@532910 532910 commented Feb 12, 2024

No description provided.

@roman-khimov
Copy link
Member

It should be updated, not deleted.

@532910
Copy link
Contributor Author

532910 commented Feb 12, 2024

#110

Either it should be up to date, or it can be deleted.

@roman-khimov
Copy link
Member

OK, after discussion with @smallhive we agreed to drop it for now, but to deal with #110 we also need to update Docker configs.

@smallhive smallhive requested a review from roman-khimov March 1, 2024 08:47
@roman-khimov
Copy link
Member

Docker is still not updated.

@532910 532910 force-pushed the remove-outdated-env branch from 9600529 to 7398478 Compare March 5, 2024 04:03
Copy link

codecov bot commented Mar 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 3.09%. Comparing base (858fcfa) to head (425fd8f).
Report is 8 commits behind head on master.

Additional details and impacted files
@@          Coverage Diff           @@
##           master    #115   +/-   ##
======================================
  Coverage    3.09%   3.09%           
======================================
  Files          18      18           
  Lines         970     970           
======================================
  Hits           30      30           
  Misses        938     938           
  Partials        2       2           

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

@roman-khimov
Copy link
Member

Please update README then, it still mentions Docker.

@532910 532910 force-pushed the remove-outdated-env branch from 7398478 to f0f7cda Compare March 5, 2024 20:54
@roman-khimov
Copy link
Member

Still mentions Docker. We also have targets in Makefile.

@532910 532910 force-pushed the remove-outdated-env branch from f0f7cda to 653acec Compare March 6, 2024 12:16
@532910 532910 force-pushed the remove-outdated-env branch from 653acec to 425fd8f Compare March 6, 2024 20:24
@roman-khimov roman-khimov merged commit 396dbdd into master Mar 6, 2024
14 checks passed
@roman-khimov roman-khimov deleted the remove-outdated-env branch March 6, 2024 20:32
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