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

Correction dette technique sur les fonctions de type prédicats ("is_xyz?" -> "xyz?") #3827

Merged
merged 41 commits into from
Mar 21, 2024

Conversation

thbar
Copy link
Contributor

@thbar thbar commented Mar 20, 2024

Voir:

Il faudra passer en revue tranquillement... c'est assez rébarbatif !

Changelogs

Liste des symboles modifiés

  • is_active?
  • is_admin?
  • is_archived?
  • is_community_resource?
  • is_current?
  • is_documentation?
  • is_documentation_format?
  • is_final_state?
  • is_format?
  • is_gbfs?
  • is_geojson?
  • is_gtfs?
  • is_gtfs_flex?
  • is_gtfs_outdated
  • is_gtfs_rt?
  • is_human?
  • is_jsonschema?
  • is_known_schema?
  • is_link_to_folder?
  • is_mailing_list?
  • is_mine?
  • is_netex?
  • is_ods_resource?
  • is_producer?
  • is_real_time?
  • is_real_time_public_transit?
  • is_resource_gtfs?
  • is_same?
  • is_same_resource?
  • is_siri?
  • is_siri_lite?
  • is_tableschema?
  • is_user_in_followers?
  • is_valid_type?
  • is_voie?
  • is_zip?

@thbar thbar added the dette technique Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité label Mar 21, 2024
@thbar thbar changed the title [WIP] Refactoring de tous les calls en "is_" Refactoring de tous les calls en "is_" Mar 21, 2024
@thbar thbar changed the title Refactoring de tous les calls en "is_" Dette technique sur les fonctions de type prédicats ("is_xyz?") Mar 21, 2024
@thbar thbar changed the title Dette technique sur les fonctions de type prédicats ("is_xyz?") Correction dette technique sur les fonctions de type prédicats ("is_xyz?" -> "xyz?") Mar 21, 2024
@thbar thbar marked this pull request as ready for review March 21, 2024 08:18
@thbar thbar requested a review from a team as a code owner March 21, 2024 08:18
@thbar
Copy link
Contributor Author

thbar commented Mar 21, 2024

@etalab/transport-tech si vous faites de la review, je peux suggérer de cloner en local et de tester un peu, voire de faire un peu de "grep", car c'est pas exclu que j'ai raté quelque chose

Copy link
Member

@AntoineAugusti AntoineAugusti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça me parait bon.

Script, pas d'erreurs de compilation et tests qui passent semble être suffisamment rassurant

@thbar
Copy link
Contributor Author

thbar commented Mar 21, 2024

Allez on y go, merci @AntoineAugusti ! Et content que cette PR ne vieillisse pas, elle serait pénible à mettre à jour.

@thbar thbar added this pull request to the merge queue Mar 21, 2024
thbar referenced this pull request Mar 21, 2024
Co-authored-by: Thibaut Barrère <[email protected]>
Merged via the queue into master with commit 91f7085 Mar 21, 2024
4 checks passed
@thbar thbar deleted the credo-predicate-function-names branch March 21, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dette technique Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants