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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
prelude: Add missing feature gate for dispatching::repls
import
#776
Conversation
@WaffleLapkin, it seems that everything is ok with this PR. We can merge it and release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, however out master
branch is currently broken (we've accidentally merged dev
into it 馃う馃徎), so we should fix it first
# TODO: prolly move it to a separate step? | ||
- name: Check with no default features | ||
uses: actions-rs/cargo@v1 | ||
with: | ||
command: check | ||
args: --no-default-features |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer if this went in with the CI refactor, but it's fine.
README.md
Outdated
<img src="./media/teloxide-logo.png" width="250"/> | ||
<img src="https://github.com/teloxide/teloxide/raw/master/media/teloxide-logo.png" width="250"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's the reason for this?...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No logo here: https://crates.io/crates/teloxide.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, right, that makes sense. Although this should be changed in dev
, I think. Since only there the media
change happened (well, ignoring the accidental merge.)
a50872e
to
7a0c70e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've cleaned up the git history. LGTM.
7a0c70e
to
02398e3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've cleaned up the git history.
Orwell's turning in his grave...
looks like I missed another instance of the #770 issue when I worked on #771... 馃檲
This PR fixes a similar issue in the
prelude
module :)