Allow raw identifier for field arguments

[zaksabeast]

Fixes #786.

This pull request allows raw identifiers to be used for field arguments, and updates multi_args_descr_trailing_comma to include a trailing comma.

[tyranron]

@zaksabeast thank you!

Would you be so kind to re-check that for GraphQL interfaces and add appropriate tests to integration_tests/juniper_tests/ too?

[zaksabeast]

@tyranron Sure thing!

I'd be happy to do the re-check and add integration tests.

Could you please elaborate on the ask to re-check GraphQL interfaces? I'm not super familiar with Juniper and might be misunderstanding the ask, so correct me if I'm wrong, but it looks like arguments can't currently be used with methods of a trait using the graphql_interface proc macro.

Would adding a test to integration_tests/juniper_tests/src/codegen/derive_object_with_raw_idents.rs make sense, or would a new file be preferred?

[tyranron]

@zaksabeast

it looks like arguments can't currently be used with methods of a trait using the graphql_interface proc macro.

Hmmm, right. I didn't figure it out. So, #[graphql_interface] doesn't have such problem at all.

Would adding a test to integration_tests/juniper_tests/src/codegen/derive_object_with_raw_idents.rs make sense, or would a new file be preferred?

It would be better to add mod raw_arguments to the integration_tests/juniper_tests/src/codegen/impl_object.rs, like fallible does.

Thanks!

[zaksabeast]

Perfect, thank you for the help!

[tyranron]

@zaksabeast thanks! 🍻