reem / stainless

Organized, flexible testing framework for Rust
MIT License
455 stars 29 forks source link

Broken with nightly as of 2018-02-28 #90

Open svenstaro opened 6 years ago

svenstaro commented 6 years ago

I currently get this on git master and nightly of 2018-02-28:

error[E0599]: no associated item named `Inherited` found for type `syntax::codemap::Spanned<syntax::ast::VisibilityKind>` in the current scope
   --> /home/svenstaro/.cargo/git/checkouts/stainless-0b08235192b57707/f89f435/src/generate.rs:135:18
    |
135 |             vis: ast::Visibility::Inherited,
    |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^ associated item not found in `syntax::codemap::Spanned<syntax::ast::VisibilityKind>`

error[E0599]: no associated item named `Inherited` found for type `syntax::codemap::Spanned<syntax::ast::VisibilityKind>` in the current scope
   --> /home/svenstaro/.cargo/git/checkouts/stainless-0b08235192b57707/f89f435/src/generate.rs:179:18
    |
179 |             vis: ast::Visibility::Inherited,
    |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^ associated item not found in `syntax::codemap::Spanned<syntax::ast::VisibilityKind>`

error[E0599]: no associated item named `Public` found for type `syntax::codemap::Spanned<syntax::ast::VisibilityKind>` in the current scope
   --> /home/svenstaro/.cargo/git/checkouts/stainless-0b08235192b57707/f89f435/src/generate.rs:228:47
    |
228 |         let super_glob = cx.item_use_glob(sp, ast::Visibility::Public, vec![cx.ident_of("super")]);
    |                                               ^^^^^^^^^^^^^^^^^^^^^^^ associated item not found in `syntax::codemap::Spanned<syntax::ast::VisibilityKind>`
svenstaro commented 6 years ago

ping @reem @ujh

Deedasmi commented 6 years ago

Are we to consider stainless broken and unmaintained?

svenstaro commented 6 years ago

I mailed both maintainers directly months ago without an answer so yes, the project is broken and unmaintained.

svenstaro commented 6 years ago

I think it's time for a proper fork. Are you up for maintaining that?

utkarshkukreti commented 6 years ago

Since you're discussing about forking, I'd like to mention a package I wrote quite a while ago which is very similar to / inspired by stainless but has some additional features like declaring functions / consts / statics outside before/after and supports passing a string to describe blocks: https://github.com/utkarshkukreti/speculate.rs. It was just updated to support the latest nightly thanks to @gilescope (https://github.com/utkarshkukreti/speculate.rs/pull/4).

It doesn't support the given and when aliases but I can add them if anyone needs it. Migrating to speculate should be straightforward:

  1. Replace before_each with before
  2. Replace after_each with after
  3. Change all inner describe blocks to pass a string and remove the !.
  4. Change the outermost describe! to speculate!.

I'm probably missing some things, so let me know if you run into any problems.

reem commented 6 years ago

Hey all, sorry for the communication delay, unfortunately I don't have time to maintain stainless any more. I would be amenable to adding more maintainers if others are interested in doing the work, so that people can continue to use the library if they find it helpful.

svenstaro commented 6 years ago

@reem can you add me as a maintainer on here and on crates.io? My mail is svenstaro@gmail.com on both. I'm an Arch Linux developer and am a pretty active open source dude in case you need any credibility. You could also transfer the repo to my GitHub account so that I don't have to bug you in case I want to add some integrations to it.

@utkarshkukreti that looks pretty dope. I'll check that out.

ms-ati commented 6 years ago

Is this transfer of maintainership of stainless happening @svenstaro @reem?

If not, in the interests of being good open source citizens, would you accept PRs to mark this as a dead project and direct to speculate instead, so that future folks have an easier time figuring out which BDD project is still alive?

svenstaro commented 6 years ago

Honestly, at this point, I'm no longer interested in maintaining stainless. I switched over to speculate.rs and I'm happy with that.

ms-ati commented 6 years ago

Oh for good grief @svenstaro @reem. Then could some one please, for the love and compassion for all folks new to Rust and its ecosystem, amend the README to mark stainless as dead and recommend that folks searching for the best project and community for rspec-inspired BDD in Rust check out speculate.rs instead?

Genuine question: is it acceptable to the Rust community for newcomers to have to ask for this?

By comparison, in my experience it's considered common courtesy in the Ruby ecosystem (among others) that when the most highly visible project in a given niche dies, they make BIG VISIBLE NOTICES referring newcomers to the most likely replacements.

Would you accept a pull request to do this?

reem commented 6 years ago

Yes I would accept a pull request directing people to a suitable and actively maintained replacement. Unfortunately I don’t really have any time right now to do maintenance, so if nobody else wants to do it I think that would be the best course of action. On Fri, Jun 22, 2018 at 5:13 PM Marc Siegel notifications@github.com wrote:

Oh for good grief @svenstaro https://github.com/svenstaro @reem https://github.com/reem. Then could some one please, for the love and compassion for all folks new to Rust and its ecosystem, amend the README to mark this as dead and recomment checking out stainless instead?

Genuine question: is it acceptable to the Rust community for newcomers to have to ask for this?

By comparison, in my experience it's considered common courtesy in the Ruby ecosystem (among others) that when the most highly visible project in a given niche dies, they make BIG VISIBLE NOTICES referring newcomers to the most likely replacements.

Would you accept a pull request to do this?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/reem/stainless/issues/90#issuecomment-399614304, or mute the thread https://github.com/notifications/unsubscribe-auth/AEhn3YG1CdDnxbyCbRH3MGEB3NEr1BSwks5t_YglgaJpZM4SXFrB .

PSeitz commented 6 years ago

@reem You can fill me in as maintainer (pascal.seitz@gmail.com), I maintain a fork anyway https://github.com/PSeitz/stainless. You can also transfer the repo, this would make things easier.