Fix#16 Remove .vscode

[mahakbansal2019]

Description

Remove .vscode

Fixes #16

Type of Change:

• Code

Code/Quality Assurance Only

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• My PR follows the style guidelines of this project
• I have performed a self-review of my own code or materials

Code/Quality Assurance Only

• My changes generate no new warnings

[mahakbansal2019]

@mtreacy002 Could you please review this.
Thanks :)

[mtreacy002]

This looks good to me, @mahakbansal2019 . Thanks for contributing. We just need to wait for one more reviewer to approve this PR before we can merge it 😉

[mtreacy002]

By the way, @mahakbansal2019, did you see my notes on issue #19? I need t move it across to backend since I mistakenly placed it in frontend. If you are still interested, here's the link for the same isse (issue#55) on backend.

[mtreacy002]

This looks good, @mahakbansal2019 . Thanks for contributing. 😁

[mtreacy002]

By the way, @mahakbansal2019 , can you please use the format available on the repository for this? Contributors are required to use the given template for consistency across the project. You can copy paste the template on to this PR description above to adopt the correct format. You can find the template for the PR here

[mtreacy002]

I'm yet to update the frontend Wiki to match its backend repo. so, the mistake is mine that you didn't use the PR template. But if you could update it by modifying the PR description here, I would really appreciate your help 👍

[mtreacy002]

Just a question, @mahakbansal2019 . when you open the Pull Request, did it come with a template? It should have come automatically.

If so, please fill up the template where it is required. And please do not remove the subheadings so to maintain consistency across AnitaB projects. It will be a good learning experience for you to bring across different open source projects in the future 😄

[mahakbansal2019]

@mtreacy002 Sorry for not following the correct template.
I have updated the description.
Hope it's fine now :)
Will do the required updates if needed :)

[mtreacy002]

Perfect.... It looks much better now (more descriptive in terms of explaining the purpose of the PR). Thank you for the update, @mahakbansal2019

[meenakshi-dhanani]

You will also have to delete the .vscode directory that is already committed. .gitignore will ensure henceforth it doesn't get committed. But for the already committed one, can you delete it?

[mahakbansal2019]

I have deleted the .vscode directory.
Thanks, @meenakshi-dhanani for the review :)

[meenakshi-dhanani]

Thank you @mahakbansal2019 ! Looks good to me. Thank you so much for contributing.
Since you have picked up a first timer issue, for the other issues, it would be nice if you involve the other community members :) That's also a way to contribute. And a very fulfilling one 😄

[meenakshi-dhanani]

@mtreacy002 Feel free to merge if it looks good

[mtreacy002]

Thanks, @meenakshi-dhanani . But I think only @anitab-org/bridgeintech-maintainers can merge a PR. The merge button is disabled on my side since I'm not authorised to merge 😁

[meenakshi-dhanani]

But I think you can approve, I can do the merging if necessary

[mtreacy002]

Great work, @mahakbansal2019 👍 .