Closed AdityaVerma32 closed 2 years ago
Code Climate has analyzed commit 02392930 and detected 0 issues on this pull request.
View more on Code Climate.
This pull request generated screenshots of many common pages in the running app. You should be able to download and view them here: https://github.com/publiclab/plots2/suites/8789946120/artifacts/399330951
Hi @AdityaVerma32, can you add a link to the issue?
One thing that can help to get started is to make sure you've included a link back to the original issue you're solving, in the format fixes #0000 (for example)
Merging #11488 (0239293) into main (55dd755) will increase coverage by
2.75%
. The diff coverage isn/a
.:exclamation: Current head 0239293 differs from pull request most recent head 47cbc97. Consider uploading reports for the commit 47cbc97 to get more accurate results
Looks like this issue has been resolved in another PR. Closing now. Thank you!
I have changed the var to let in (https://github.com/publiclab/plots2/tree/main/spec/javascripts)/inline_grids_spec.js file as stated in the issue.
Fixes #0000
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test
@publiclab/reviewers
for help, in a comment below