Skip to content

Conversation

@bhartijoshi04
Copy link
Contributor

No description provided.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to Our repository.🎊 Thank you so much for taking the time to point this out.

@bhartijoshi04 bhartijoshi04 changed the title stackoverflow_analysis_2021 stackoverflow_analysis_2021/2022 May 19, 2024
@bhartijoshi04 bhartijoshi04 changed the title stackoverflow_analysis_2021/2022 stackoverflow_analysis_2021&2022 May 19, 2024
Copy link
Member

@sanjay-kv sanjay-kv left a comment

Choose a reason for hiding this comment

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

Could you let me know the issue assigned.

  • the thing is we don't need different ipynb file. look at the backlog folder. many people has created different ipynb file. The main idea is add on to the existing ipynb file.

Copy link
Member

@sanjay-kv sanjay-kv left a comment

Choose a reason for hiding this comment

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

Kindly move to backlog folder

@bhartijoshi04
Copy link
Contributor Author

I am unable to do so, my forked repository isn't up to date. I did try to fork again but I am unable to see the backlog folder in forked repository
However, I did commit changes in the main file.
You can go through that @sanjay-kv .

@sanjay-kv
Copy link
Member

you gotta do git pull in commant promt.

@bhartijoshi04
Copy link
Contributor Author

@sanjay-kv Now you can check, I've added the 2021.CSV file to the data folder and updated the analysis notebook

@bhartijoshi04 bhartijoshi04 changed the title stackoverflow_analysis_2021&2022 stackoverflow_analysis_2021 May 20, 2024
@sanjay-kv
Copy link
Member

Data folder is for only CSV files.
Push the main files to backlog, I will review it and merge with main file later

Copy link
Member

@sanjay-kv sanjay-kv left a comment

Choose a reason for hiding this comment

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

sorry, there are changes required. i commented the same.

@bhartijoshi04
Copy link
Contributor Author

I've updated the analysis file as per your previous analysis with the markdown included,

@bhartijoshi04
Copy link
Contributor Author

Data folder is for only CSV files. Push the main files to backlog, I will review it and merge with main file later

Yeah, I've added the csv file only in the data folder and the ipynb file of stackoverflow_analysis, I've updated the 2021_stack's EDA into it.

@sanjay-kv
Copy link
Member

can you fix the confliclts files, then I will merge it

@bhartijoshi04
Copy link
Contributor Author

can you fix the confliclts files, then I will merge it

Can u tell me the reason for the conflict, I am unable to see through it.

@sanjay-kv sanjay-kv removed the level1 label May 21, 2024
@sanjay-kv
Copy link
Member

As you spent more time on this I upgraded the labels to level 2

@bhartijoshi04
Copy link
Contributor Author

I'll add it separately in the backlog folder, you can review it from there after then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants