Comments (18)
We are going to roll out the config files soon, which will hopefully allow to fine-tune and configure tools not just on the per-repo but even on the per-directory basis (we all know how there are often different measures of good practices in tests and elsewhere) — it's in the works, likely next week.
As for the UI configuration, that's going to be rolled out with the UI rework which depends on the front-end contractor so far. Would it be more convenient for you to put the config file(s) in the target repo, or to make adjustments via the UI?
from metal-rs.
Hi! I'm coming with the news that we have rolled in the configuration files. You can consult the documentation here.
So far, we don't have many tools attached, so if you have an idea for what you might want or need, we'll be happy to hear.
We are also considering moving towards using GitHub Checks functionality, at least in part.
Any feedback is highly appreciated.
from metal-rs.
@gingerDevilish thanks for checking in!
Monocodus was a forcing function for us to move the codebases to be rust-formatted. It was a concern earlier that if we do this, it would be hard to keep it formatted: either we'd land non-formatted changes and try to clean them up, or we fail on CI, which is annoying to users. Monocodus strikes me as a great alternative - it instantly suggests changes, basically resolving the concern for us.
Is there something in the list that would be of use to your team?
We haven't used anything in this list, I think, but overall we'll appreciate anything in this direction.
The very basic tool that a lot of project use is clippy
. We use it too, and we'd love to see it integrated with Monocodus. It would need to be configurable though: one of the simple use cases is only blocking on Clippy errors, in which case we wouldn't want to spam the warnings with Monocodus.
Also, here's our public issues tracker in case you'd like to throw in some suggestions.
Great, looking forward to complain a lot! :)
from metal-rs.
Hi @kvark!
I hope you're doing well.
We rolled an update just today, and here's what we got now:
- Support of GH's multiline suggestions — now suggestions from our autofix tools should become much less confusing. Besides, as we have reworked the algorithm of file processing, more fixes are suggested and fewer dumb missuggestions will occur.
- We have temporarily disabled some of the checkers for C, Python, and JS, because of them being deemed too spammy. They'll return later after a major rework, removing annoying notifications because of just one extra line above a threshold. We're seeking to only point out real prospective problems, not to be meddley.
Coming next:
- Clippy support!
- Then, we're going to focus on adding more checkers
Let us know if you have any issues — and stay tuned!
from metal-rs.
Hi!
I hope you're doing fine.
We have just deployed an update — this time this is clippy support. To enable it, you need to update your config to version 1.1.0, plus include native clippy configs in your projects if you need more fine-tuning.
So far we've got a spammy mode where we output all the comments we can place in the diff, and a succinct mode which only tells you that issues were found, so you may want to run the tool on your computer and fix them. Please let us know if you need any additional configuration options with respect to the monocodus output!
We're also looking for the best way to convert the clippy
advice to automated suggestions while retaining explanations, so you can expect more automation soon.
Happy coding, and have a great summer!
from metal-rs.
Hi!
I'm happy to announce that we've just released suggestions integration for clippy — so now you can apply the fixes in a couple of clicks :)
You don't need to do anything this time — no configuration changes are required.
Please let us know if you see any issues, or if you have any other requests — and tell your friends about us if you like the ways monocodus is trying to help you
Happy coding, and stay tuned!
from metal-rs.
We'd be happy to try this out on some projects. Thank you for the heads up!
from metal-rs.
I'm surprised you need "We ask for read and write access" though. Isn't the goal to generate PR suggestions? How does this require "write"?
from metal-rs.
Could you please show me where you have seen that? I'm pretty sure we did not set the requested permissions for the write access, as we are currently stalled on the features that would require that, and discussing the further course of action. It must be a textual mistake on our side.
That said, the team has been discussing approaches to comments on formatting (it's clear that it is not especially feasible to point out such things on the line-to-line basis), and we have some feature requests for automated reformatting triggered by a respective command/mention, which would indeed require the write access, but it's not the case at the moment.
from metal-rs.
Okay, I re-checked it with the team, and here's what is actually happening.
While we don't request write access to your code, we do request it to pull requests:
This is necessary in order to post review commens and thus required for the core bot functionality.
It still seems we have to reformulate that phrase so that there is more clarity.
from metal-rs.
This is what scares me off:
By signing up, you are agreeing to our Terms of Service and Privacy Policy. We ask for read and write access. Verified email on GitHub is required.
If that's truly about only PR writes, it's very important to be clear about this in one of the first sentences that the user sees.
from metal-rs.
I signed up now. So it looks like Monocodues comes with a number of services built in. The first thing I would want is being able to configure (on a per-repository basis) which services are enabled, and which are disabled. Otherwise, there is going to be a storm of suggestions we don't want yet. I'm not seeing this ability anywhere in the UI...
from metal-rs.
Yes, putting configs in the repo is totally fine with us.
from metal-rs.
Hey, I'm coming back for a quick check-in.
Did you try to set up a config? If so, did any difficulties arrive?
from metal-rs.
It was fairly straightforward so far, we disabled rustfmt and JS passes on some of the projects. Thank you!
from metal-rs.
Okay, good to know
If you happen to wish for a UX tweak, find a bug, or have a feature request, feel free to reach out!
from metal-rs.
I sent 2 emails to support on April 20th without any sign of an answer.
from metal-rs.
Hey, just a quick check-in again. How has it been lately?
We have improved some wordings a while back during past few weeks, and are currently working on the architecture revamp to allow more substantial checks to be added. This summer, we are considering the integration of some of these verification tools. Is there something in the list that would be of use to your team?
Also, here's our public issues tracker in case you'd like to throw in some suggestions.
from metal-rs.
Related Issues (20)
- Derive PartialEq on MTlScissorRect
- Naming conventions HOT 5
- [MTLIGRenderPipelineState setLabel:]: unrecognized selector sent to instance
- fatal assert when calling ArgumentDescriptor::set_access HOT 2
- Document function availability on platforms
- Publishing new version HOT 2
- Debug groups and signposts on render passes
- `MTLPixelFormat` and related enumeration types are unsound? HOT 1
- Don't use auto-release pool in string-related methods
- Using metal-cpp for auto-generating bindings HOT 2
- Investigate use of objc_retainAutoreleasedReturnValue HOT 6
- Make auto-release pools visible HOT 1
- Different behaviour using Metal with Swift in Xcode vs Rust HOT 2
- Incorrect buffer value in compute example HOT 2
- Export MTLAccelerationStructure
- computation performs slower than cpu version in benchmark HOT 3
- Including `metal-rs` on `cargo.toml` results in a compile error due to `cocoa` HOT 1
- trouble with xcode shader debugger locating source code HOT 1
- Question about mesh shaders HOT 3
- Updating version on crates.io HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from metal-rs.